Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add JSX transpiler to dashboard build process (estimated: 1) #1502

Closed
muloem opened this issue Nov 24, 2015 · 7 comments
Closed

Add JSX transpiler to dashboard build process (estimated: 1) #1502

muloem opened this issue Nov 24, 2015 · 7 comments

Comments

@muloem
Copy link
Member

muloem commented Nov 24, 2015

As part of the introduction of react components in the dashboard, we need to be able to compile JSX to JS code that will be embedded in different parts of the ember dashboard code. This requires including JSX transpiler tools in our current build process.

Child of #2796

@muloem muloem self-assigned this Nov 25, 2015
@muloem muloem modified the milestone: 1.9.4 Dec 1, 2015
@muloem muloem modified the milestones: 1.9.5, 1.9.4 Dec 28, 2015
@rumca rumca modified the milestones: 1.9.6, 1.9.5 Feb 22, 2016
@rumca rumca modified the milestones: 1.9.7, 1.9.6 Apr 4, 2016
@muloem muloem modified the milestones: 1.9.7, 1.9.8 Apr 8, 2016
@janagombitova janagombitova modified the milestones: 1.9.9, 1.9.8, Priority 1 May 11, 2016
@janagombitova janagombitova changed the title Add JSX transpiler to dashboard build process Add JSX transpiler to dashboard build process (estimated: 1) May 11, 2016
@janagombitova janagombitova modified the milestone: Priority 1 Nov 2, 2016
finnfiddle added a commit that referenced this issue Mar 6, 2019
finnfiddle added a commit that referenced this issue Mar 6, 2019
finnfiddle added a commit that referenced this issue Mar 6, 2019
@ghost ghost assigned finnfiddle Mar 6, 2019
@janagombitova
Copy link
Contributor

janagombitova commented Mar 14, 2019

I hope this is the last issue to be found:

  • answers to repeated groups do not show in the Inspect or Monitoring tabs. The repetitions show but the answers are blank. In Maps all works fine.

Screen Shot 2019-03-14 at 16 06 09

@kymni
Copy link
Contributor

kymni commented Mar 15, 2019

@janagombitova seems the reason this particular response does not display is because the question is type option but the values submitted are free text (maybe was initially text then changed type after submitting?). Flow therefore tries to look for text property in the response but not finding it returns null hence the blank. Issue is therefore not webpack related but we still need to catch such cases

@janagombitova
Copy link
Contributor

@kymni that you for checking. Thas it very odd. I will try to reproduce it on a production instance and create a separate issue. Then all webpack issues should be handled. I did not find anything more.

finnfiddle added a commit that referenced this issue Mar 27, 2019
finnfiddle added a commit that referenced this issue Mar 27, 2019
finnfiddle added a commit that referenced this issue Mar 29, 2019
finnfiddle added a commit that referenced this issue Mar 29, 2019
finnfiddle added a commit that referenced this issue Mar 29, 2019
* develop: (48 commits)
  [#3026]Stop updating the backend.
  [#3024] release notes
  Update bootstrap-deploy.sh
  [#3022]Avoid NPE if no user found.
  [#3018] release notes
  [#3018]Initial version of release notes.
  [#2969] Remove whitespace
  [#2971]Extract constants. Remove trailing whitespace.
  [#3017] Add NSS package
  [#3017] Add NSS package
  [#3017] Add NSS package
  [#3017] Add NSS package
  [#2970]Remove trailing whitespace.
  [#2970]Formatting...
  [#2802] unused function
  [#2802] do it the ember way
  [#2694] Remove backends configuration file
  [#2696] Refactor cascade node deletion
  [#2694] Remove backends deployment code
  [#2971]Stop calling taks on backend. Remove unused stuff.
  ...
finnfiddle added a commit that referenced this issue Mar 29, 2019
muloem added a commit that referenced this issue Mar 29, 2019
[#1502] Switch UI build tools from ruby based to webpack
kymni added a commit that referenced this issue Apr 1, 2019
[#1502] Add missing variable to generate version package
@finnfiddle finnfiddle added this to the 1.9.45 R... R... milestone Apr 4, 2019
@finnfiddle finnfiddle reopened this Apr 4, 2019
finnfiddle added a commit that referenced this issue Apr 4, 2019
finnfiddle added a commit that referenced this issue Apr 4, 2019
kymni pushed a commit that referenced this issue Apr 4, 2019
kymni pushed a commit that referenced this issue Apr 4, 2019
finnfiddle added a commit that referenced this issue Apr 5, 2019
kymni added a commit that referenced this issue Apr 5, 2019
finnfiddle added a commit that referenced this issue Apr 8, 2019
@valllllll2000
Copy link
Contributor

everything works as expected

@kymni kymni closed this as completed in 8e483c3 Apr 15, 2019
finnfiddle added a commit that referenced this issue Apr 17, 2019
* develop: (46 commits)
  [#3069] fix tail command
  #3035 removed git hooks config
  #3035 made post merge git script silent
  #3035 remove ruby build fix
  #3035 devserver script removed typo
  capturing also the output of npm install
  #3035 made npm tasks parallel in devserver script
  #3035 removed unused file assoc. with ruby build and fixes
  [#3046] release notes update
  [#3046] release notes for 1.9.45
  remove ruby build altogether #3035
  [#3057] do not require users.scss
  [#2772]Explicit scopes. Tabs to spaces. Split long method.
  #1502 added script for compiling users page css
  #1502 added users css file
  travis slack notifications #3052
  #1502 added back rake build command
  [#1502] node has to be installed
  [#1502] fix package.json dir
  #1502 dockerfile npm install fixes
  ...

# Conflicts:
#	Dashboard/app/js/lib/main-public.js
#	Dashboard/app/js/lib/views/devices/assignments-list-view.jsx
#	Dashboard/package-lock.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants