-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
i18nification #549
i18nification #549
Conversation
40% of work done |
@andrew can you review the changes. I translated the majority of the code except usage which is not directly linked from front page. If it's ok for you, it can be merged |
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.
Just one small change otherwise LGTM
@@ -7,7 +7,7 @@ class Owner < ApplicationRecord | |||
scope :created_after, ->(date) { where('created_at > ?', date) } | |||
scope :updated_after, ->(date) { where('updated_at > ?', date) } | |||
|
|||
scope :kind, ->(kind) { where(kind: kind) } | |||
enum kind: { user: 'user', organization: 'organization' } |
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 still need the scope of kind here for the owners controller
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 reverted the removal of the kind scope
…nly scope to filter
implement #546