Rename "webcompat-reporter" directory to "report-site-issue"

RESOLVED FIXED

Status

enhancement
P3
normal
RESOLVED FIXED
7 months ago
28 days ago

People

(Reporter: miketaylr, Assigned: marian.meireles)

Tracking

(Blocks 1 bug)

unspecified
Dependency tree / graph

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

7 months ago
Just to reduce confusion when we talk about our 2 systems addons, let's rename the "webcompat-reporter" to "report-site-issue".

(more than a few people have been confused at the difference between the "webcompat" and "webcompat-reporter" addons)
Comment hidden (obsolete)
Reporter

Updated

7 months ago
Flags: needinfo?(miket)
I'm guessing we'll also be moving over any localization string-bundles to new directories, and renaming the about:config prefs as well, yes?
Reporter

Comment 3

4 months ago

(In reply to Thomas Wisniewski [:twisniewski] (PTO Dec 24-26, 31-2nd) from comment #2)

I'm guessing we'll also be moving over any localization string-bundles to
new directories, and renaming the about:config prefs as well, yes?

Yeah, probably. I guess we want to rename everything here: https://searchfox.org/mozilla-central/search?q=webcompat-reporter

Reporter

Comment 4

4 months ago

Something you want to work on Mariana?

Flags: needinfo?(marian.meireles)
Assignee

Comment 5

4 months ago

(In reply to Mike Taylor [:miketaylr] from comment #4)

Something you want to work on Mariana?

Yeah, sure! :)

Flags: needinfo?(marian.meireles)
Reporter

Updated

4 months ago
Assignee: nobody → marian.meireles
Attachment #9039561 - Attachment is obsolete: true
Blocks: 1524523

This seems to have been granted review, is there a reason it hasn't landed yet?

Flags: needinfo?(miket)
Flags: needinfo?(marian.meireles)
Reporter

Comment 9

2 months ago

Hm... I think we just forgot. :(

@Mariana, do you think you can re-base the patch? I think Tom's work on about:compat might have created some conflicts. Then let's get it landed!

Flags: needinfo?(miket)
Assignee

Comment 10

2 months ago

This: https://phabricator.services.mozilla.com/D17890#change-r65ycskz7t2N was the only thing I had to change.

How do I proceed now? Should I ping florian and Pike again to review my work?

Flags: needinfo?(marian.meireles)
Reporter

Comment 11

2 months ago

Nope, no need for a re-review, IMO.

Could you please squash that second fixup commit into the first?

Flags: needinfo?(marian.meireles)
Assignee

Comment 12

29 days ago

Sorry for the delay.
I think it's finally correct!

Flags: needinfo?(marian.meireles) → needinfo?(miket)
Reporter

Updated

29 days ago
Type: defect → enhancement
Flags: needinfo?(miket)
Reporter

Updated

29 days ago
Flags: needinfo?(miket)
Reporter

Comment 13

29 days ago

Tom, can you double check and help land this?

Flags: needinfo?(miket) → needinfo?(twisniewski)

Comment 14

29 days ago
Pushed by twisniewski@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ba4736a11504
Rename webcompat-reporter directory to report-site-issue r=florian,Pike

Comment 15

28 days ago
bugherder
Status: NEW → RESOLVED
Last Resolved: 28 days ago
Resolution: --- → FIXED
Flags: needinfo?(twisniewski)
You need to log in before you can comment on or make changes to this bug.