Skip to content
This repository was archived by the owner on May 22, 2021. It is now read-only.

Add Pontoon URL to README#380

Merged
pdehaan merged 2 commits intomozilla:masterfrom
pdehaan:readme-pontoon
Aug 2, 2017
Merged

Add Pontoon URL to README#380
pdehaan merged 2 commits intomozilla:masterfrom
pdehaan:readme-pontoon

Conversation

@pdehaan
Copy link
Copy Markdown
Contributor

@pdehaan pdehaan commented Aug 2, 2017

Words are not great. Suggestions/commits welcome!

r? @flodolo

@pdehaan pdehaan requested a review from flodolo August 2, 2017 00:04
@pdehaan
Copy link
Copy Markdown
Contributor Author

pdehaan commented Aug 2, 2017

Looks a bit clumsy, but we could also do a custom "Pontoon" badge-esque thing at the top of the README.md file:

editing_send_readme_md_at_master_ _mozilla_send

[![Firefox Send on Pontoon](https://img.shields.io/badge/Pontoon-L20n-7bc876.svg)](https://pontoon.mozilla.org/projects/test-pilot-firefox-send/)

README.md Outdated
## Localization

_Coming soon_ (see [#57](https://github.com/mozilla/send/issues/57))
Firefox Send localizations are managed via [Pontoon](https://pontoon.mozilla.org/projects/test-pilot-firefox-send/).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a personal preference for "localization is managed", but plural works too. The badge is nice, but not sure it's effective (you would need to know what Pontoon or l20n are).

Maybe worth adding a bit more of information:

Firefox Send localization is managed via [Pontoon](https://pontoon.mozilla.org/projects/test-pilot-firefox-send/), not direct pull requests to the repository. If you want to fix a typo, add a new language, or simply know more about localization, please get in touch with the [existing localization team](https://pontoon.mozilla.org/teams/) for your language, or Mozilla’s [l10n-drivers](https://wiki.mozilla.org/L10n:Mozilla_Team#Mozilla_Corporation) for guidance.

@pdehaan
Copy link
Copy Markdown
Contributor Author

pdehaan commented Aug 2, 2017

@flodolo Yeah, yours is way better, thanks! Pushing new commit.

Copy link
Copy Markdown
Collaborator

@flodolo flodolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. We should probably use this as a boiler plate for all other projects 👍

@pdehaan pdehaan merged commit 49d5782 into mozilla:master Aug 2, 2017
@pdehaan
Copy link
Copy Markdown
Contributor Author

pdehaan commented Aug 2, 2017

OK, I think I got all the current/active Test Pilot experiments (except Tab Center, which has been retired), as well as Screenshots and Activity Stream. I'll leave it to the repo owners to merge as they see fit.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants