SmartPhone Code Transition: Difference between revisions

Line 32: Line 32:
|-
|-
| rowspan="1" |  [transition] Browser: Remove reference and logos relating to Firefox
| rowspan="1" |  [transition] Browser: Remove reference and logos relating to Firefox
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252193 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252193 Bug ID] 1232193
| 1. Remove all references to Firefox OS
| 1. Remove all references to Firefox OS
2. Change browser icon to not be firefox logo  
2. Change browser icon to not be firefox logo  
Line 58: Line 58:
|-
|-
| rowspan="1" |  Remove FTU app
| rowspan="1" |  Remove FTU app
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252427 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252427 Bug ID] 1252427
| Keeping the FTU app around and maintain it during the transition period isn't a high priority. We should move the code and tests to a separate repository so it doesn't get lost. We might want to leave the 'hooks' to create a custom FTU app in the system app but we won't maintain ours going forward. This also includes the tutorial parts.
| Keeping the FTU app around and maintain it during the transition period isn't a high priority. We should move the code and tests to a separate repository so it doesn't get lost. We might want to leave the 'hooks' to create a custom FTU app in the system app but we won't maintain ours going forward. This also includes the tutorial parts.
My assumption is that users don't use any functionality since they can do everything in the settings app afterwards.
My assumption is that users don't use any functionality since they can do everything in the settings app afterwards.
Line 67: Line 67:
|-
|-
| rowspan="1" |  Remove mozPay
| rowspan="1" |  Remove mozPay
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252570 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252570 Bug ID] 1252570
| The Marketplace is removing payments support [1], so we can get rid of all mozPay related code.
| The Marketplace is removing payments support [1], so we can get rid of all mozPay related code.


Line 73: Line 73:
| Acceptance Criteria to be added
| Acceptance Criteria to be added
|-
|-
| rowspan="1" |  Unknown
| rowspan="1" |  Remove MobileID API
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252573 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252573 Bug ID] 1252573
| Unknown
| This API was added mostly because of Firefox Hello requirements. Unfortunately, the Firefox Hello client for FxOS didn't last long and I don't think we have any other consumer for this API. So we can probably remove it safely.
| Unknown
| MobileID API removed
|-
|-
| rowspan="1" |  Remove "improve Firefox OS" from settings
| rowspan="1" |  Remove "improve Firefox OS" from settings
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252773 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252773 Bug ID] 1252773
| Remove "improve Firefox OS" from settings
| Remove "improve Firefox OS" from settings
| No reference to Firefox OS in settings
| No reference to Firefox OS in settings
|-
|-
| rowspan="1" |  Rename Firefox OS to B2G OS
| rowspan="1" |  Rename Firefox OS to B2G OS
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252776 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252776 Bug ID] 1252776
| Rename  - settings - device information - “about Firefox OS" - to About B2G OS - including all subtext.
| Rename  - settings - device information - “about Firefox OS" - to About B2G OS - including all subtext.
| "about Firefox OS" renamed to "about B2G OS" in device information under settings
| "about Firefox OS" renamed to "about B2G OS" in device information under settings
|-
|-
| rowspan="1" |  Review all legal statements under settings
| rowspan="1" |  Review all legal statements under settings
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252778 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252778 Bug ID] 1252778
| Review content under:
| Review content under:
  a. settings - device info - your rights
  a. settings - device info - your rights
Line 98: Line 98:
|-
|-
| rowspan="1" |  Remove Firefox OS logos from startup sequence
| rowspan="1" |  Remove Firefox OS logos from startup sequence
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252779 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252779 Bug ID] 1252779
| Remove all logos from the device startup sequence that shows and refers to Firefox OS.
| Remove all logos from the device startup sequence that shows and refers to Firefox OS.
| No Firefox OS logos in the device startup sequence
| No Firefox OS logos in the device startup sequence
|-
|-
| rowspan="1" |  Remove current location for OTA lookup
| rowspan="1" |  Remove current location for OTA lookup
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252789 Bug ID]
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252789 Bug ID] 1252789
| Remove the current location pointer for OTS search but still leave the OTA option in the settings so future possible OTA hosting services can just put their URL in there for usage.
| Remove the current location pointer for OTS search but still leave the OTA option in the settings so future possible OTA hosting services can just put their URL in there for usage.
| Current location pointer for OTS search removed
| Current location pointer for OTS search removed
|-
|-
| rowspan="1" |  Integrate nginx
| rowspan="1" |  Integrate nginx
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1253600 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1253600 Bug ID] 1253600
| Following bug 1252371 we are relying on an http daemon. Let's use one with a bit more features to allow future work (https, multiple domains)
| Following bug 1252371 we are relying on an http daemon. Let's use one with a bit more features to allow future work (https, multiple domains)
| Acceptance criteria to be entered
| Acceptance criteria to be entered
|-
|-
| rowspan="1" |  Build system changes for the transitioned Gaia
| rowspan="1" |  Build system changes for the transitioned Gaia
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254281 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254281 Bug ID] 1254281
| This is to prepare for a chrome:// served Gaia. Instead of packaging each app independently in an application.zip file, we will package them all together, and put the shared/ alongside the apps.
| This is to prepare for a chrome:// served Gaia. Instead of packaging each app independently in an application.zip file, we will package them all together, and put the shared/ alongside the apps.
The directory tree will look like:
The directory tree will look like:
Line 130: Line 130:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the audiochannel code
| rowspan="1" |  Remove the dependency on apps status in the audiochannel code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254282 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254282 Bug ID] 1254282
| It should be chrome only now. See http://mxr.mozilla.org/mozilla-central/source/dom/system/gonk/AudioChannelManager.cpp#157 for a starting point of things to fix.
| It should be chrome only now. See http://mxr.mozilla.org/mozilla-central/source/dom/system/gonk/AudioChannelManager.cpp#157 for a starting point of things to fix.
| Acceptance criteria
| Acceptance criteria
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the NFC code
| rowspan="1" |  Remove the dependency on apps status in the NFC code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254283 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254283 Bug ID] 1254283
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 142: Line 142:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the Bluetooth code
| rowspan="1" |  Remove the dependency on apps status in the Bluetooth code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254284 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254284 Bug ID] 1254284
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 149: Line 149:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the Downloads api code
| rowspan="1" |  Remove the dependency on apps status in the Downloads api code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254285 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254285 Bug ID] 1254285
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 156: Line 156:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the RequestSync api cod
| rowspan="1" |  Remove the dependency on apps status in the RequestSync api cod
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254286 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254286 Bug ID] 1254286
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 163: Line 163:
|-
|-
| rowspan="1" |  Remove the dependency on apps status in the EngineeringMode api code
| rowspan="1" |  Remove the dependency on apps status in the EngineeringMode api code
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254287 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254287 Bug ID] 1254287
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
| Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.


Line 170: Line 170:
|-
|-
| rowspan="1" |  Remove all uses of APP_STATUS_CERTIFIED and APP_STATUS_PRIVILEGED
| rowspan="1" |  Remove all uses of APP_STATUS_CERTIFIED and APP_STATUS_PRIVILEGED
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254292 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1254292 Bug ID] 1254292
| On top of the webidl gated instances, there are direct use of these constants that we need to update:
| On top of the webidl gated instances, there are direct use of these constants that we need to update:


Line 183: Line 183:
|-
|-
| rowspan="1" |  Testing / Build Related Tasks
| rowspan="1" |  Testing / Build Related Tasks
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252482 Bug ID]  
| [https://bugzilla.mozilla.org/show_bug.cgi?id=1252482 Bug ID] 1252482
| Meta for tracking build and test related tasks for fxos sunset program
| Meta for tracking build and test related tasks for fxos sunset program
|
|