[Settings] developer info in Application Permissions always links to http://github.com/mozilla-b2g/gaia/

VERIFIED FIXED

Status

Firefox OS
Gaia::Settings
P3
normal
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: MattN, Assigned: MattN)

Tracking

({b2g-testdriver})

unspecified
All
Gonk (Firefox OS)
b2g-testdriver

Firefox Tracking Flags

(blocking-basecamp:+)

Details

Attachments

(1 attachment)

developerName.href never gets changed from http://github.com/mozilla-b2g/gaia/ in apps/settings/js/apps.js

Clicking on the developer name always links to http://github.com/mozilla-b2g/gaia/
Created attachment 679020 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/6226

Pointer to Github pull-request
Attachment #679020 - Flags: review?(21)
This was because the code only looked at the data-href for external URLs but that was not getting updated when switching apps in the permission screen. While fixing this I realized it also affected the developer's URL line as well.
Summary: [Settings] developer name in Application Permissions always links to http://github.com/mozilla-b2g/gaia/ → [Settings] developer info in Application Permissions always links to http://github.com/mozilla-b2g/gaia/
Comment on attachment 679020 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/6226

Mostly Etienne's code. Let's redirect the review.
Attachment #679020 - Flags: review?(21) → review?(etienne)
Hey Matthew, commented on the pull request, just a small change needed.

Thanks for the fix!
Attachment #679020 - Flags: review?(etienne) → review+
blocking-basecamp: ? → +
Priority: -- → P3
https://github.com/mozilla-b2g/gaia/commit/a8e3fd1481263a705a546544ef20b4005cc5b9cf
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Keywords: verifyme
QA Contact: jsmith
Component: Gaia::Apps Management → General
Component: General → Gaia

Updated

5 years ago
Component: Gaia → Gaia::Settings
Looks good. Verified on 11/26.
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.