The default bug view has changed. See this FAQ.

about:apps page on FxAndroid has a marketplace link pointing to https://marketplace.mozilla.org, not https://marketplace.firefox.com

RESOLVED FIXED in Firefox 25

Status

()

Firefox for Android
Web Apps
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jsmith, Assigned: jsx)

Tracking

Trunk
Firefox 25
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=mfinkle@mozilla.com][lang=js])

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
Created attachment 772390 [details]
Marketplace Link

Build: Firefox 25 Nightly
Device: Galaxy Nexus
OS: Android 4.2

STR

1. Go to about:apps
2. Long press the marketplace link

Expected

The link should point to https://marketplace.firefox.com.

Actual

The link points to https://marketplace.mozilla.org.
http://mxr.mozilla.org/mozilla-central/source/mobile/android/app/mobile.js#473
Whiteboard: [good first bug][mentor=mfinkle@mozilla.com][lang=js]
I would like to get started with contributing code. I've no previous experience contributing code to Mozilla. Can I take this and work on this?
Sure thing.
Assignee: nobody → saurabh
Thanks. As I said, this is my first time. So this might be a stupid question. Is there a git repo that I can clone and make the modification and send in a patch?
I suggest reading through https://wiki.mozilla.org/Mobile/Fennec/Android and first getting a build up and running that you can install to your device or emulator. Feel free to hop on to #mobile on irc.mozilla.org to chat with us and if you need assistance.

https://developer.mozilla.org/en-US/docs/Developer_Guide

Building Fennec: https://wiki.mozilla.org/Mobile/Fennec/Android#Building_Fennec
Getting the source: https://wiki.mozilla.org/Mobile/Fennec/Android#Getting_the_source
Thanks for the detailed info Aaron. I'll do that.
I've made the change and created a pull request in Github.

https://github.com/mozilla/mozilla-central/pull/15

Will this be ok? Or do I need to upload a patch?
Created attachment 773941 [details] [diff] [review]
Attaching patch
We use Mercurial not the Github mirror, so the patch will need to be Mercurial based https://developer.mozilla.org/en-US/docs/Mercurial_FAQ
Attachment #773941 - Flags: review?(mark.finkle)
Is there anyway I can clone the just the 'mobile' directory from the repo rather than the entire mozilla-central repo?
Attachment #773941 - Flags: review?(mark.finkle) → review+
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/9f3556f19859
Keywords: checkin-needed
Is there any further action required from me? Because I got an email saying patch approved.
(In reply to Saurabh Nair [:jsx] from comment #12)
> Is there any further action required from me? Because I got an email saying
> patch approved.

Nope. The patch was landed on mozilla-inbound and will make it's way to mozilla-central (and Nightly) soon.

Thanks for the patch Saurabh. We hope you find another bug to work on next. Drop by #mobile on IRC if you want to talk about next bug opportunities.
(In reply to Mark Finkle (:mfinkle) from comment #13)
> Drop by #mobile on IRC if you want to talk about next bug opportunities.

Sure, I'll Mark. Thanks for all the help, Mark and Aaron.
https://hg.mozilla.org/mozilla-central/rev/9f3556f19859
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
You need to log in before you can comment on or make changes to this bug.