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 Ruby on Rails v6 example which uses Webpacker #197

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

amree
Copy link

@amree amree commented Nov 21, 2020

Add an example for Ruby on Rails v6 which uses webpacker to handle its javascript. The current example is the older version which uses an asset pipeline, so, a different example is needed for the newer version of Ruby on Rails.

p/s: I can create two commits to make it clearer, one to create the project and the other is for adding TradingView's integration. I don't think we should update the current Ruby on Rails example. I think lots of people are still using that version.

Add example checklist

  • Any commit does not contain the charting library's code
  • Addresses an existing issue: #00000
  • The example is added to README.md
  • The example is self-sufficient and contains only necessary files/changes

Bug fix checklist

  • Any commit does not contain the charting library's code
  • Addresses an existing issue: #00000

Add an example for Ruby on Rails v6 which uses webpacker to handle its
javascript. The current example is the older version which uses an asset
pipeline, so, a different example is needed for the newer version of
Ruby on Rails.
@amree amree force-pushed the add-ruby-on-rails-v6-example branch from 0c83c49 to 43cfad0 Compare November 21, 2020 11:53
Copy link
Contributor

@timocov timocov left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Just few comments.

@@ -0,0 +1,17 @@
# README
Copy link
Contributor

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

yes

- Copy `charting_library/charting_library.js` into `app/javascript/packs/charting_library/charting_library.js`
- Copy `datafeeds/udf/dist/*.js` into `app/javascript/packs/datafeeds/`
- Copy `charting_library/*.html` into `public/charting_library/`
- Copy `charting_library/bundles` into `public/charting_library/bundles`
Copy link
Contributor

Choose a reason for hiding this comment

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

It seems that we have to run yarn before run the app?

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, is it possible to migrate to npm instead?

Choose a reason for hiding this comment

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

Of course

@@ -0,0 +1 @@
yvu8/Qw+RUlkD8CltYTm6GlzVF/3SVimkO6JG/Yp2mC4XL9JYU+QEGAwxfiL6YsIlyepj4EiKd42Qy76bWA6HfRtmktjS9pCfd6fwEyEJjdKa75V08Ui2xrwvHj42IKguf25v8/IW+iFMbm2wnXiYFuA0aOHQNhCXOf8PoCxqDJMlZcsJOYrGrmtm8EFwIJGOU8RYkUw1nWw9nL/MngNPFPQnvzyTcQUdEWqUUZwXKpaDMHWxncjZjn2acsLM8uvZwT5kByhjWp9rudhmFJh7k0zYwgMcPskTCv6rJ7t1xliMTbABzImAFcIMeUVv0/jdmgQBO6GmFZpfliD6Bw8Bu9PmTLXbivyVi4Iasl7PDoFEqNwO5xwUu+wx25p4NGFOHTx/dcrTOEdi7a4MCjmDs6QTAfcgIkcupR9--jmXkdcXEWEVobOEJ--iZuQgWRu9plT8TqWJTS0vg==
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this auto-generated? How to (re-)generate it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants