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

Add Pontoon URL to README #380

Merged
merged 2 commits into from
Aug 2, 2017
Merged

Add Pontoon URL to README #380

merged 2 commits into from
Aug 2, 2017

Conversation

pdehaan
Copy link
Collaborator

@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
Collaborator 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
@@ -32,7 +32,7 @@ $ redis-server /usr/local/etc/redis.conf

## 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
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
Collaborator Author

pdehaan commented Aug 2, 2017

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

Copy link
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
Collaborator 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