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

React on Rails ReasonHelloWorld component #1215

Closed
wants to merge 2 commits into from

Conversation

ljuster
Copy link

@ljuster ljuster commented May 8, 2019

REASON/REACT
- bsconfig
- bsb make and clean commands in build
- ReasonHelloWorld.re component
- added component to client registration file
RAILS
- add route
- add hyperlink on main page
- view file


This change is Reviewable

    - bsconfig
    - bsb make and clean commands in build
    - ReasonHelloWorld.re component
    - added component to client registration file
  RAILS
    - add route
    - add hyperlink on main page
    - view file
@@ -4,6 +4,7 @@ import 'es5-shim';
import ReactOnRails from 'react-on-rails';

import HelloWorld from '../components/HelloWorld';
import HelloWorldReason from '../components/HelloWorldReason.bs';
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@justin808 as is, it won't recognize HelloWorldReason.bs component.

clientRegistration.jsx?209f:1 Uncaught Error: Cannot find module "../components/HelloWorldReason.bs"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just not how you integrate the bs into the JS. The JS code needs the compiled file to import.

@justin808
Copy link
Member

@ljuster how's this going? We need CI to pass.

@justin808
Copy link
Member

@Judahmeek check this one out. Would be great to get this merged.

@justin808
Copy link
Member

@Judahmeek @behraaangm can we get this merged into spec/dummy?

@behraaangm
Copy link
Contributor

Hey @justin808, I wanna make a new branch from this work.
the package.json in spec/dummy/client is should be removed, right?

@justin808
Copy link
Member

Hey @justin808, I wanna make a new branch from this work.

Sure!

the package.json in spec/dummy/client is should be removed, right?

Yes!

@justin808
Copy link
Member

See #1399

@justin808 justin808 closed this Nov 2, 2021
gscarv13 added a commit to gscarv13/react_on_rails that referenced this pull request Nov 3, 2021
@gscarv13 gscarv13 mentioned this pull request Nov 3, 2021
justin808 pushed a commit that referenced this pull request Dec 30, 2021
* Cherry-pick commits from #1215
* Add ReScript dependencies and configuration
* Remove Reason file, Replace Reason occurrences by ReScript
* Add ReScript component, update yarn.lock at spec/dummy/
  Create file to import compiled ReScript components
* Add rake task to build ReScript when running rake assets:precompile
* Add JavaScript dependency required to build for production
* Run prettier and add *.bs.js files to .prettierignore
* Add return at the end of the line package.json
* Bump prettier, concurrently and prettier-eslint-cli
  add files to .prettierignore
* Update bsconfig.json
superdev9082 added a commit to superdev9082/react_on_rails that referenced this pull request Feb 16, 2023
* Cherry-pick commits from shakacode/react_on_rails#1215
* Add ReScript dependencies and configuration
* Remove Reason file, Replace Reason occurrences by ReScript
* Add ReScript component, update yarn.lock at spec/dummy/
  Create file to import compiled ReScript components
* Add rake task to build ReScript when running rake assets:precompile
* Add JavaScript dependency required to build for production
* Run prettier and add *.bs.js files to .prettierignore
* Add return at the end of the line package.json
* Bump prettier, concurrently and prettier-eslint-cli
  add files to .prettierignore
* Update bsconfig.json
Web-Go-To added a commit to Web-Go-To/react-with-rails that referenced this pull request Mar 19, 2023
* Cherry-pick commits from shakacode/react_on_rails#1215
* Add ReScript dependencies and configuration
* Remove Reason file, Replace Reason occurrences by ReScript
* Add ReScript component, update yarn.lock at spec/dummy/
  Create file to import compiled ReScript components
* Add rake task to build ReScript when running rake assets:precompile
* Add JavaScript dependency required to build for production
* Run prettier and add *.bs.js files to .prettierignore
* Add return at the end of the line package.json
* Bump prettier, concurrently and prettier-eslint-cli
  add files to .prettierignore
* Update bsconfig.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants