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

Use lazy translation lookups for the user_blocks partials #1827

Merged
merged 1 commit into from
Apr 18, 2018

Conversation

gravitystorm
Copy link
Collaborator

Lazy translations are easier to read, to write, and easier to deal with when refactoring things like partial names, controller methods and controller names.

This is just a first gentle step in a wider project.

@Nikerabbit as per #1289 this PR will involve changing translation keys on Translatewiki, if it is merged.

@Nikerabbit
Copy link
Contributor

Thanks for the heads up.

@tomhughes
Copy link
Member

As you've only changed the english messages I don't think this will affect @Nikerabbit as such, but it does mean those messages will have to be translated again.

@Nikerabbit
Copy link
Contributor

Nikerabbit commented Apr 18, 2018

After the PR is merged, I will apply the renames to also translations so they don't actually need to be translated again.

@tomhughes tomhughes merged commit 736ff9b into openstreetmap:master Apr 18, 2018
@tomhughes
Copy link
Member

OK. I've merged it now if you want to do your magic.

@gravitystorm
Copy link
Collaborator Author

It looks like everything was renamed on translatewiki - thanks @Nikerabbit !

@gravitystorm gravitystorm deleted the lazy_translations branch April 25, 2018 02:58
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