Closed
Bug 1881215
Opened 2 years ago
Closed 1 year ago
rename mobile/android/focus-android to just mobile/android/focus (post move to central)
Categories
(Focus :: General, task, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: jcristau, Assigned: adhingra)
References
(Blocks 1 open bug)
Details
Attachments
(1 obsolete file)
It feels weird/unnecessary to have that "focus-android" path in an android directory.
Updated•2 years ago
|
Product: Fenix → Focus
Updated•2 years ago
|
Severity: -- → N/A
Updated•2 years ago
|
Priority: -- → P3
Updated•1 year ago
|
No longer blocks: monorepo-cleanup
Severity: N/A → S4
Comment 1•1 year ago
|
||
I believe the blocking meta bug was removed in error.
Blocks: monorepo-cleanup
| Assignee | ||
Comment 2•1 year ago
|
||
Updated•1 year ago
|
Assignee: nobody → adhingra
Status: NEW → ASSIGNED
Comment 3•1 year ago
•
|
||
I would really suggest not to do this.
It might seem like a simple change, but I don't think the downstream consequences are easy to identify.
- There is RelEng automation used to import strings from the
android-l10nrepository. - There is automation in GitHub to expose the strings from the
mozilla-firefox/firefoxrepository intoandroid-l10n. - There is Pontoon and other automation expecting to find strings in the current path (example for Italian, notice the URL).
And there is probably more relying on the existing path.
If you think this work is required, this might be large enough to require planning and time on my team's roadmap.
In any case, it needs investigation before any change lands, and that's not something I have bandwidth to do in the coming weeks.
Updated•1 year ago
|
Attachment #9494504 -
Attachment description: WIP: Bug 1881215 - Rename mobile/android/focus-android to just mobile/android/focus → Bug 1881215 - Rename mobile/android/focus-android to just mobile/android/focus
Updated•1 year ago
|
Attachment #9494504 -
Attachment is obsolete: true
Comment 4•1 year ago
|
||
Based on comment 3, let's close this. Re-open if you feels strongly about it.
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•