Skip to content

Fixes #778: Remove Gaia#779

Merged
TheoChevalier merged 3 commits into
mozfr:masterfrom
TheoChevalier:remove-gaia
Jul 29, 2016
Merged

Fixes #778: Remove Gaia#779
TheoChevalier merged 3 commits into
mozfr:masterfrom
TheoChevalier:remove-gaia

Conversation

@TheoChevalier
Copy link
Copy Markdown
Member

No description provided.

@mention-bot
Copy link
Copy Markdown

@TheoChevalier, thanks for your PR! By analyzing the annotation information on this pull request, we identified @flodolo, @pascalchevrel and @Thegennok to be potential reviewers

@flodolo
Copy link
Copy Markdown
Collaborator

flodolo commented Jul 29, 2016

Looks like there are more tests to cleanup/update (Health.php)?

@flodolo
Copy link
Copy Markdown
Collaborator

flodolo commented Jul 29, 2016

@TheoChevalier
Copy link
Copy Markdown
Member Author

I think they aren’t obsolete (yet) https://transvision.mozfr.org/?recherche=%7B%7B&repo=central&sourcelocale=en-US&locale=fr&search_type=strings Switched to desktop since it’s what makes most sense currently.


; URL to l10n web service. Wrap the value between quotes if the URL contains special characters like ~
l10nwebservice = "https://l10n.mozilla-community.org/~flod/mozilla-l10n-query/"
l10nwebservice = "https://flod.org/mozilla-l10n-query/"
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.

Note: remember to restore the value, and also to drop the ~flod/ part

@flodolo
Copy link
Copy Markdown
Collaborator

flodolo commented Jul 29, 2016

Everything looks great to me. Should we merge and open a pull request on-hold to restore the URL?

@TheoChevalier
Copy link
Copy Markdown
Member Author

yeah, lets :)

@TheoChevalier TheoChevalier merged commit e593ec3 into mozfr:master Jul 29, 2016
@TheoChevalier TheoChevalier mentioned this pull request Jul 29, 2016
@TheoChevalier TheoChevalier deleted the remove-gaia branch July 31, 2016 03:16
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