Closed
Bug 917738
Opened 11 years ago
Closed 11 years ago
features added in bug 875415 (?) aren't being detected in manifest/app
Categories
(Marketplace Graveyard :: Validation, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
2013-09-24
People
(Reporter: eviljeff, Unassigned)
References
Details
if an app asks for systemXHR in the manifest (or sets the mozSystem property on an XMLHttpRequest - whichever way is appropriate to detect it) then the SystemXHR buchet should be preselected. None of the packaged apps that I've spot checked have this buchet enabled now so a mop-up script will need to be run to add those buchets to the effected apps.
Comment 1•11 years ago
|
||
None of the "new" buchets are automatically detected because we haven't added any of them to the validator yet.
Component: Developer Pages → Validation
Reporter | ||
Comment 2•11 years ago
|
||
which 'new' buchets are missing?
Comment 3•11 years ago
|
||
Everything from this down: https://github.com/mozilla/zamboni/blob/master/mkt/constants/features.py#L212
Reporter | ||
Comment 4•11 years ago
|
||
:( updating summary, etc.
Severity: normal → major
Summary: SystemXHR permission isn't being feature detected in manifest → features added in bug 875415 (?) aren't being detected in manifest/app
Updated•11 years ago
|
Priority: -- → P1
Target Milestone: --- → 2013-09-24
Comment 5•11 years ago
|
||
This is not a P1, since the only platforms which support installing web apps all either do or do not support any of the new buchets (for the time being).
Priority: P1 → P2
Comment 6•11 years ago
|
||
We added quite a few: https://github.com/mozilla/fireplace/commit/f2131d53fb
Reporter | ||
Comment 7•11 years ago
|
||
true - while packaged apps for Android is turned off its not hitting any apps now. But the longer its not implemented the larger the backlog of apps without the correct buchets will be.
Comment 8•11 years ago
|
||
https://github.com/mozilla/app-validator/commit/6e03b59434cfd66db68e94f1e0aafd5d2af29f98
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•