-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Convert admin RABL views to Jbuilder #2146
Conversation
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.
Looks legit
backend/solidus_backend.gemspec
Outdated
@@ -31,6 +31,7 @@ Gem::Specification.new do |s| | |||
s.add_dependency 'jquery-ui-rails', '~> 5.0.0' | |||
s.add_dependency 'font-awesome-rails', '~> 4.0' | |||
s.add_dependency 'kaminari', '>= 0.17', '< 2.0' | |||
s.add_dependency 'jbuilder' |
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.
We should set version numbers
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.
👍 done
b451351
to
1891ac9
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.
I’m still confused by the weird “dot parentheses” syntax, but the tests pass, so ¯\_(ツ)_/¯
1891ac9
to
4424d2f
Compare
When you see anything that looks like this: |
Yeah, JBuilder uses this (maybe too) cleverly as a shorthand for multiple attributes. I replaced a few spots where it didn't make sense to use this over explicitly declaring the attributes on separate lines. |
RABL was removed from the project on version 2.4.0 (See solidusio#2147 solidusio#2146). This updates the responder template to match with Jbuilder and updates the README file.
RABL was removed from the project on version 2.4.0 (See solidusio#2147 solidusio#2146). This updates the responder template to match with Jbuilder and updates the README file.
No description provided.