Closed
Bug 1395956
Opened 7 years ago
Closed 7 years ago
Update gecko.readthedocs links to the new firefox-source-docs.mozilla.org location
Categories
(Toolkit :: Telemetry, enhancement, P2)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: chutten, Assigned: kausam2015, Mentored)
Details
(Whiteboard: [good first bug][lang=rst][lang=js][lang=c++][lang=java])
Attachments
(1 file, 2 obsolete files)
13.21 KB,
patch
|
chutten
:
review+
|
Details | Diff | Splinter Review |
According to searchfox there are 15 remaining links in mozilla-central that are pointing at gecko.readthedocs.io: http://searchfox.org/mozilla-central/search?q=gecko.readthedocs&path= gecko.readthedocs.io is no longer being updated (firefox source docs having exceeded the size limits of readthedocs). We are instead now hosting it on a domain mozilla controls at firefox-source-docs.mozilla.org This bug is about changing those gecko.readthedocs.io links in the source tree to point to their new locations somewhere within firefox-source-docs.mozilla.org This might not simply be a substitution of firefox-source-docs.mozilla.org for gecko.readthedocs.io as ReadTheDocs had "en/latest" in a lot of links and firefox-source-docs.mozilla.org doesn't. Be sure to test the new links to make sure they point at valid locations.
Reporter | ||
Updated•7 years ago
|
Mentor: chutten
Comment 1•7 years ago
|
||
Where can I access the files because I am new to this field could you please help me? also Where to start?
Attachment #8903989 -
Flags: review?(chutten)
Comment 4•7 years ago
|
||
Comment on attachment 8903989 [details] [diff] [review] patchfile Review of attachment 8903989 [details] [diff] [review]: ----------------------------------------------------------------- This looks good, thanks! There are only some minor trailing spaces that need to be removed before we can proceed. Note that the standard commit message convention is "Bug XXX - ..." (vs. your "Bug XXX: ..."). ::: mobile/android/base/java/org/mozilla/gecko/telemetry/TelemetryBackgroundReceiver.java @@ +38,4 @@ > /** > * Receives and processes telemetry broadcasts from background services, namely Sync. > * Nomenclature: > + * - Bundled Sync Ping: a Sync Ping as documented at https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/data/sync-ping.html There is a trailing space here on this line, please remove it. ::: mobile/android/base/java/org/mozilla/gecko/telemetry/pingbuilders/TelemetryCorePingBuilder.java @@ +36,4 @@ > /** > * Builds a {@link TelemetryOutgoingPing} representing a core ping. > * > + * See https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/data/core-ping.html There is a trailing space here on this line, please remove it. ::: mobile/android/base/java/org/mozilla/gecko/telemetry/pingbuilders/TelemetrySyncPingBundleBuilder.java @@ +26,4 @@ > > /** > * Responsible for building a Sync Ping, based on the telemetry docs: > + * https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/data/sync-ping.html More trailing space.
Attachment #8903989 -
Flags: review?(chutten) → feedback+
Updated•7 years ago
|
Assignee: nobody → kausam2015
Priority: -- → P2
Attachment #8904321 -
Flags: review?(gfritzsche)
Comment 6•7 years ago
|
||
(In reply to Cosm from comment #2) > I am working on this bug. Sorry I am Totaly new to it Could help me in finding where should I start ?
Comment 7•7 years ago
|
||
Comment on attachment 8904321 [details] [diff] [review] patchfile.patch Chris is back today, so i'm moving this review over to him.
Attachment #8904321 -
Flags: review?(gfritzsche) → review?(chutten)
Comment 8•7 years ago
|
||
(In reply to ppaswanth3 from comment #6) > (In reply to Cosm from comment #2) > > I am working on this bug. > > Sorry I am Totaly new to it > Could help me in finding where should I start ? Hi! I think this is not the best medium to help you get started. Please see the information over here, it gets you started and points you to chat and other forms of getting help: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Introduction
Reporter | ||
Comment 9•7 years ago
|
||
Comment on attachment 8904321 [details] [diff] [review] patchfile.patch Review of attachment 8904321 [details] [diff] [review]: ----------------------------------------------------------------- Good work! Fix the one link and put r=chutten on the end of the first line of your commit message (a la https://developer.mozilla.org/en-US/docs/Mercurial/Using_Mercurial#Commit_Message_Conventions ), reupload it to this bug, and we'll be ready to get this code submitted. ::: devtools/docs/frontend/telemetry.md @@ +38,4 @@ > > New data types have been added over the years, so it's quite feasible that some of our probes are not the most suitable nowadays. > > +There's more information about types (and telemetry in general) on [this page](https://developer.mozilla.org/en-US/docs/Mozilla/Performance/Adding_a_new_Telemetry_probe) and [this other page](https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/index.html). The second link should be https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/collection/index.html
Attachment #8904321 -
Flags: review?(chutten) → review+
Assignee | ||
Comment 10•7 years ago
|
||
Thank you Chris, for the pointing out the correction.
Attachment #8903989 -
Attachment is obsolete: true
Attachment #8904321 -
Attachment is obsolete: true
Attachment #8904633 -
Flags: review?(chutten)
Reporter | ||
Updated•7 years ago
|
Attachment #8904633 -
Flags: review?(chutten) → review+
Reporter | ||
Comment 11•7 years ago
|
||
I have marked this bug for checkin, so your code should reach the tree shortly and then be included in either tomorrow's or the next day's Nightly build of Firefox. Congratulations! Do you need any help finding another patch to work on, or would you like to run through the list on Bugs Ahoy (https://www.joshmatthews.net/bugsahoy/?internals=1) to find something you find interesting?
Keywords: checkin-needed
Comment 12•7 years ago
|
||
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/dadf26985c95 Update gecko.readthedocs links to the new firefox-source-docs.mozilla.org location. r=chutten
Keywords: checkin-needed
Comment 13•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/dadf26985c95
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•