-
Notifications
You must be signed in to change notification settings - Fork 7
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
pkp/pkp-lib#9910 Use UI Language in Web Feeds #3
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.
@bozana I just think it's better to keep the country information of the locale (if available).
Hi @jonasraoni, I considered the comments, could you please take another look? |
@jonasraoni, what about this solution? |
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 just left some notes, nothing important =]
0ab13fd
to
32afe65
Compare
Hi @jonasraoni, I was waiting with this PR till I have finished the integration of Weblate locales. Could you please take a look at it now? Thanks a lot! |
s. pkp/pkp-lib#9910