Error link from about:support leads to about:policies#active instead of about:policies#errors

VERIFIED FIXED in Firefox 63

Status

()

P1
normal
VERIFIED FIXED
7 months ago
6 months ago

People

(Reporter: emilghitta, Assigned: lschwalf, Mentored)

Tracking

(Blocks: 1 bug, {good-first-bug})

63 Branch
Firefox 64
good-first-bug
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox-esr60 unaffected, firefox61 unaffected, firefox62 unaffected, firefox63 verified, firefox64 verified)

Details

Attachments

(2 attachments)

(Reporter)

Description

7 months ago
[Affected versions]:
Firefox 63.0a1 (BuildId:20180903100454).

[Affected platforms]:
Windows 10 64bit.
macOS 10.13.4
Ubuntu 16.04 64bit.

[Preconditions]:
Enter an invalid policy.

[Steps to reproduce]:
1. Launch Firefox.
2. Access the about:support page.
3. Click the "Error" link from the "Enterprise Policies" section.

[Expected result]:
The about:policies#errors is displayed.

[Actual result]:
The about:policies#active is displayed.

[Notes]
For further information regarding this issue please observe the attached screencast.
Mentor: felipc
Keywords: good-first-bug
Priority: -- → P5
(Assignee)

Updated

6 months ago
Assignee: nobody → lschwalfenberg
Status: NEW → ASSIGNED

Updated

6 months ago
Blocks: 1493190
Comment on attachment 9011123 [details]
Bug 1488241 - Error link from about:support leads to about:policies#active instead of about:policies#errors. r?felipe

:Felipe Gomes (needinfo me!) has approved the revision.
Attachment #9011123 - Flags: review+
Thanks for your contribution, Luke! Can you fix the eslint error that showed up (missing semi-colon) and submit an updated patch? The code change is correct, so that's all that's missing.

If you're are interested, there are two other good-first-bugs related to about:policies that we really would like that a new contributor picked up. They are:

  - Bug 1485109 - Simpler, but of higher importance
  - Bug 1488411 - Slightly more complex

Comment 4

6 months ago
Pushed by paolo.mozmail@amadzone.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ff4f20dd4858
Error link from about:support leads to about:policies#active instead of about:policies#errors. r=felipe

Updated

6 months ago
Priority: P5 → P1
(Assignee)

Comment 5

6 months ago
(In reply to :Felipe Gomes (needinfo me!) from comment #3)
> Thanks for your contribution, Luke! Can you fix the eslint error that showed
> up (missing semi-colon) and submit an updated patch? The code change is
> correct, so that's all that's missing.
> 
> If you're are interested, there are two other good-first-bugs related to
> about:policies that we really would like that a new contributor picked up.
> They are:
> 
>   - Bug 1485109 - Simpler, but of higher importance
>   - Bug 1488411 - Slightly more complex

Thanks Felipe! 

Fixed the eslint error, and updated the change set. 

I'm looking forward to contributing more to Firefox. I might take a look at those other bugs at some point, but at the moment I'm ramping up to work with :paolo and my other UCOSP team members to work on updates we're making to "about:config".

Cheers, 
Luke

Comment 6

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ff4f20dd4858
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago
status-firefox64: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Please request beta uplift when you get a chance.
Flags: needinfo?(felipc)
Comment on attachment 9011123 [details]
Bug 1488241 - Error link from about:support leads to about:policies#active instead of about:policies#errors. r?felipe

Approval Request Comment
[Feature/Bug causing the regression]: about:policies
[User impact if declined]: Link in about:support to about:policies is not ideal
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: not yet
[Needs manual test from QE? If yes, steps to reproduce]: QE will verify it
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: simple changing the link href
[String changes made/needed]: none
Flags: needinfo?(felipc)
Attachment #9011123 - Flags: approval-mozilla-beta?
Comment on attachment 9011123 [details]
Bug 1488241 - Error link from about:support leads to about:policies#active instead of about:policies#errors. r?felipe

Low risk patch for a P1 bug, uplift approved for 63 beta 10, thanks.
Attachment #9011123 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(Reporter)

Comment 10

6 months ago
This issue is verified fixed using Firefox 64.0a1 (BuildId:20180924220042) on Windows 10 64bit, macOS 10.13.4 and Ubuntu 16.04 64bit.
Status: RESOLVED → VERIFIED
status-firefox64: fixed → verified

Comment 11

6 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/e71d11110002
status-firefox63: affected → fixed
(Reporter)

Comment 12

6 months ago
This issue is verified fixed using Firefox 63.0b10 (BuildId:20180927143327) on Windows 10 64bit, macOS 10.13.4 and Ubuntu 16.04 64bit.

Thanks for fixing this!
status-firefox63: fixed → verified
You need to log in before you can comment on or make changes to this bug.