-
Notifications
You must be signed in to change notification settings - Fork 771
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
base: master
Are you sure you want to change the base?
Conversation
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.
0c83c49
to
43cfad0
Compare
There was a problem hiding this 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please make readme file similar to https://github.com/tradingview/charting-library-examples/tree/master/ruby-on-rails?
There was a problem hiding this comment.
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` |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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== |
There was a problem hiding this comment.
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?
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
Bug fix checklist