Skip to content
This repository has been archived by the owner on Jun 21, 2022. It is now read-only.

Fixing bug 1371143 #202

Merged
merged 2 commits into from Jun 13, 2017
Merged

Fixing bug 1371143 #202

merged 2 commits into from Jun 13, 2017

Conversation

SphinxKnight
Copy link
Member

This one line edit should fix https://bugzilla.mozilla.org/show_bug.cgi?id=1371143.
I didn't test it again an environment but the impact is minor (only localizers using those links... I'll be glad to have many people complaining :) )

@Elchi3 no need to rush this one, it can wait, feel free to make someone else review it

Fixing the link since macros now live on GitHub and should be PR to be modified
@SphinxKnight SphinxKnight requested a review from Elchi3 June 12, 2017 18:45
@SphinxKnight
Copy link
Member Author

The first commit was only here to update my fork (no change regarding master at that time), feel free to squash while merging

Copy link
Contributor

@jwhitlock jwhitlock left a comment

Choose a reason for hiding this comment

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

👍 worked for me.

I checked out this branch, and grabbed the status page:

./manage.py scrape_document https://developer.mozilla.org/fr/docs/MDN/Doc_status/Learn/Commencer_avec_le_Web

@Elchi3 Elchi3 merged commit 0832f5c into mdn:master Jun 13, 2017
jwhitlock added a commit to mdn/kuma that referenced this pull request Jun 13, 2017
* mdn/kumascript#190 - Split WebExtAllExamples from WebExtExamples
* mdn/kumascript#196 - AddonSidebar: Cleanup themes
* mdn/kumascript#197 - AddonSidebar: Add Creating an appealing...
* mdn/kumascript#198 - remove HTML-Element_Navigation
* mdn/kumascript#199 - remove MozillaOnly
* mdn/kumascript#200 - AddonSidebar: Add Android links
* mdn/kumascript#201 - AddonSidebar: Fix relative link
* mdn/kumascript#202 - LocalizationStatusInSection: Link to repo
* mdn/kumascript#203 - SpecName, spec2: Add Pointer Events 2, etc.
* mdn/kumascript#206 - Fix Jenkins pipeline
@SphinxKnight
Copy link
Member Author

Thanks @jwhitlock @Elchi3

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.

None yet

3 participants