Bookmark "Help and Tutorials" in Mozilla Firefox folder should open https://support.mozilla.org/en-US/products/firefox instead of https://www.mozilla.org/en-US/firefox/help/ with 404 error

VERIFIED FIXED in Firefox 53

Status

()

Firefox
Bookmarks & History
VERIFIED FIXED
a year ago
3 months ago

People

(Reporter: hans, Assigned: Alan, Mentored)

Tracking

({good-first-bug})

44 Branch
Firefox 53
good-first-bug
Points:
---

Firefox Tracking Flags

(firefox53 verified, firefox54 verified)

Details

(Whiteboard: [good first bug][lang=js])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

a year ago
User Agent: Mozilla/5.0 (Windows NT 6.0; rv:44.0) Gecko/20100101 Firefox/44.0
Build ID: 20160210153822

Steps to reproduce:

I clicked Help in the Moilla Firefox bookmarks folder


Actual results:

I was taken to a Mozilla page that told me I had been misdirected. Fault announcement 404 was shown, "page not found", but it was a Mozilla page.


Expected results:

Simply, I should have been helped

Comment 1

a year ago
Yes, the bookmark should open the localized page https://support.mozilla.org/en-US/products/firefox instead of https://www.mozilla.org/en-US/firefox/help/
Status: UNCONFIRMED → NEW
Component: Untriaged → Bookmarks & History
Ever confirmed: true
Summary: Choosing the bookmark link Help in Mozilla bookmarks (https://www.mozilla.org/nl/firefox/help/) I am directed to a page that says Oops, you lost the right direction (or something like it, it was in Dutch) → Bookmark "Help and Tutorials" in Mozilla Firefox folder should open https://support.mozilla.org/en-US/products/firefox instead of https://www.mozilla.org/en-US/firefox/help/ with 404 error

Comment 2

6 months ago
Is this bug still unresolved?
(Reporter)

Comment 3

6 months ago
I do not know, I do not understand the original problem, what is Help in the bookmarksfolder? It is too long ago. Consider it resolved.

Comment 4

6 months ago
Seems to be WFM.
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → WORKSFORME
FWIW, http://searchfox.org/mozilla-central/rev/51aa673e28802fe6ea89f4793721fc55334a6ac8/browser/locales/generic/profile/bookmarks.html.in#36 still has the old URL...but https://www.mozilla.org/nl-nl/firefox/help/ and https://www.mozilla.org/nl-be/firefox/help/ correctly redirect to https://support.mozilla.org/nl/products/firefox.
Same for https://www.mozilla.org/nl/firefox/help/.

Comment 7

6 months ago
Let' use the bug to fix bookmarks.html.in then. Sounds easy enough for a first good bug.
Mentor: mak77@bonardo.net
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Whiteboard: [good first bug][lang=js]

Updated

6 months ago
Keywords: good-first-bug
(Assignee)

Comment 8

6 months ago
I would love to be assigned to this! So after reading the above comments I see that:
  1. https://www.mozilla.org/nl-nl/firefox/help/
  2. https://www.mozilla.org/nl-be/firefox/help/
  3. https://www.mozilla.org/nl/firefox/help/ 
All correctly redirect to https://support.mozilla.org/nl/products/firefox.

But, bookmarks.html.in is redirecting to the incorrect URL.

So I should have that also redirect to https://support.mozilla.org/nl/products/firefox.
(Assignee)

Comment 9

6 months ago
(In reply to ayx002 from comment #8)
> I would love to be assigned to this! So after reading the above comments I
> see that:
>   1. https://www.mozilla.org/nl-nl/firefox/help/
>   2. https://www.mozilla.org/nl-be/firefox/help/
>   3. https://www.mozilla.org/nl/firefox/help/ 
> All correctly redirect to https://support.mozilla.org/nl/products/firefox.
> 
> But, bookmarks.html.in is redirecting to the incorrect URL.
> 
> So I should have that also redirect to
> https://support.mozilla.org/nl/products/firefox.

I would like the add I have the FireFox for Desktop code base and able to build!
(Assignee)

Comment 10

6 months ago
Created attachment 8824616 [details] [diff] [review]
No need to redirect.

Comment 11

6 months ago
(In reply to Alan from comment #10)
> Created attachment 8824616 [details] [diff] [review]
> No need to redirect.

Hey Alan, thanks for looking at this. Are you using mercurial/hg or git, and if so can you put this on mozreview or put up a patch on this bug? See https://mozilla-version-control-tools.readthedocs.io/en/latest/hgmozilla/firefoxworkflow.html and https://mozilla-version-control-tools.readthedocs.io/en/latest/mozreview.html for some documentation. Do feel free to ask more questions if things are unclear. :-)
Assignee: nobody → ayx002
Mentor: gijskruitbosch+bugs@gmail.com
Status: REOPENED → ASSIGNED
(Assignee)

Comment 12

6 months ago
(In reply to :Gijs from comment #11)
> (In reply to Alan from comment #10)
> > Created attachment 8824616 [details] [diff] [review]
> > No need to redirect.
> 
> Hey Alan, thanks for looking at this. Are you using mercurial/hg or git, and
> if so can you put this on mozreview or put up a patch on this bug? See
> https://mozilla-version-control-tools.readthedocs.io/en/latest/hgmozilla/
> firefoxworkflow.html and
> https://mozilla-version-control-tools.readthedocs.io/en/latest/mozreview.
> html for some documentation. Do feel free to ask more questions if things
> are unclear. :-)

So I have Mercurial setup. I am able to create a commit, however, I am not able to push to Mozreview. When I try to push I recieve a "remote: Permission denied (publickey)." Does this mean my setup in the hg config file is incorrect? I used the setup wizard too.
Comment hidden (mozreview-request)
(Assignee)

Updated

6 months ago
Attachment #8824839 - Flags: review?(gijskruitbosch+bugs)
(Assignee)

Comment 14

6 months ago
(In reply to Alan from comment #12)
> (In reply to :Gijs from comment #11)
> > (In reply to Alan from comment #10)
> > > Created attachment 8824616 [details] [diff] [review]
> > > No need to redirect.
> > 
> > Hey Alan, thanks for looking at this. Are you using mercurial/hg or git, and
> > if so can you put this on mozreview or put up a patch on this bug? See
> > https://mozilla-version-control-tools.readthedocs.io/en/latest/hgmozilla/
> > firefoxworkflow.html and
> > https://mozilla-version-control-tools.readthedocs.io/en/latest/mozreview.
> > html for some documentation. Do feel free to ask more questions if things
> > are unclear. :-)
> 
> So I have Mercurial setup. I am able to create a commit, however, I am not
> able to push to Mozreview. When I try to push I recieve a "remote:
> Permission denied (publickey)." Does this mean my setup in the hg config
> file is incorrect? I used the setup wizard too.

Woo! Was able to create the review! So I had my [path] for SSH and not HTTPS!
(Assignee)

Updated

6 months ago
Attachment #8824839 - Flags: review?(gijskruitbosch+bugs) → review?(mak77)
(Assignee)

Updated

6 months ago
Attachment #8824839 - Flags: review?(gijskruitbosch+bugs)

Updated

6 months ago
Attachment #8824839 - Flags: review?(mak77)

Comment 15

6 months ago
mozreview-review
Comment on attachment 8824839 [details]
Fix to Bug 1253227 changing bookmarks.html.in#36 from old URL to /products/firefox

https://reviewboard.mozilla.org/r/103120/#review103772

Very nice, thanks!
Attachment #8824839 - Flags: review?(gijskruitbosch+bugs) → review+

Comment 16

6 months ago
I've pushed this to our tryserver to see if tests are happy with this - apparently changes to this file have a history of accidentally breaking tests that contain assumptions about these bookmarks...

https://treeherder.mozilla.org/#/jobs?repo=try&revision=ef15dd85e27a46700822f37acf69fe67f56a8dab

Comment 17

6 months ago
This looks like tests are fine, so I've landed it for you via autoland. Thanks again for the patch. Let me know if you want a pointer to some other bugs that might provide a follow-up challenge! :-)

Comment 18

6 months ago
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/81a4f1a55040
Fix to Bug 1253227 changing bookmarks.html.in#36 from old URL to /products/firefox r=Gijs
(Assignee)

Comment 19

6 months ago
(In reply to :Gijs from comment #17)
> This looks like tests are fine, so I've landed it for you via autoland.
> Thanks again for the patch. Let me know if you want a pointer to some other
> bugs that might provide a follow-up challenge! :-)

I would love a pointer to more bugs! Thank you so much, it's a great feeling finishing my first patch!

Comment 20

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/81a4f1a55040
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago6 months ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53

Comment 21

5 months ago
(In reply to Alan from comment #19)
> (In reply to :Gijs from comment #17)
> > This looks like tests are fine, so I've landed it for you via autoland.
> > Thanks again for the patch. Let me know if you want a pointer to some other
> > bugs that might provide a follow-up challenge! :-)
> 
> I would love a pointer to more bugs! Thank you so much, it's a great feeling
> finishing my first patch!

Great! I realized I'm not sure what programming languages you're comfortable with. For some small JS stuff, see e.g. bug 1312216. For some CSS work (assuming you have access to Windows 7 or Windows 8), see e.g. bug 1111138. Feel free to needinfo me and/or the mentor of the bug (where it isn't me) if you need more details to get going on those, or if neither of those work and you'd like me to find you something else. :-)
(Assignee)

Comment 22

5 months ago
(In reply to :Gijs from comment #21)
> (In reply to Alan from comment #19)
> > (In reply to :Gijs from comment #17)
> > > This looks like tests are fine, so I've landed it for you via autoland.
> > > Thanks again for the patch. Let me know if you want a pointer to some other
> > > bugs that might provide a follow-up challenge! :-)
> > 
> > I would love a pointer to more bugs! Thank you so much, it's a great feeling
> > finishing my first patch!
> 
> Great! I realized I'm not sure what programming languages you're comfortable
> with. For some small JS stuff, see e.g. bug 1312216. For some CSS work
> (assuming you have access to Windows 7 or Windows 8), see e.g. bug 1111138.
> Feel free to needinfo me and/or the mentor of the bug (where it isn't me) if
> you need more details to get going on those, or if neither of those work and
> you'd like me to find you something else. :-)

Thank you! I got another r+ for a bug on SeaMonkey! But I definitely want to try a bug using Python.

Comment 23

5 months ago
(In reply to Alan from comment #22)
> Thank you! I got another r+ for a bug on SeaMonkey! But I definitely want to
> try a bug using Python.

Oh, right. I don't work on much that touches python, but you could try looking at:

https://www.joshmatthews.net/bugsahoy/?py=1&unowned=1

which should have a number of mentored bugs that involve python and are currently unowned. :-)

Comment 24

5 months ago
[Bugday-20170125]
Operating System : Windows 10
Browser:
status-firefox54: --- → verified

Comment 25

5 months ago
[Bugday-20170125]
Operating System : Windows 10
Browser: Firefox Nighly

    I verified the bug. While clicking on bookmark folder, I got help from the Firefox help. Thus the bug is fixed.
Thank you
Status: RESOLVED → VERIFIED
QA Whiteboard: [good first verify]

Comment 27

3 months ago
[Bugday-20170329]
Operating System:Windows 10(64-bit)
Browser: Firefox Beta 53.0b8
status-firefox53: fixed → verified
You need to log in before you can comment on or make changes to this bug.