Closed Bug 768205 Opened 13 years ago Closed 13 years ago

Marketplace Manifest Validator fails validation for manifests containing the new 'appcache_path' attribute

Categories

(Marketplace Graveyard :: Validation, defect)

defect
Not set
normal

Tracking

(blocking-kilimanjaro:?)

RESOLVED FIXED
blocking-kilimanjaro ?

People

(Reporter: nick, Assigned: basta)

Details

From Ticket #760365 <https://bugzilla.mozilla.org/show_bug.cgi?id=760365>, the dev marketplace will fail any new app submissions that contain the new "appcache_path" attribute: Unknown property found. Error: An unknown property was found in the webapp manifest. Error: Found property: appcache_path
Assignee: nobody → mattbasta
So I see this property in this manifest — http://people.mozilla.com/~fdesre/openwebapps/app.manifest — but I don't see it in the spec: http://mozilla.github.com/webapps-spec/ Is this in line with the spec?
(In reply to Chris Van Wiemeersch [:cvan] from comment #1) > So I see this property in this manifest — > http://people.mozilla.com/~fdesre/openwebapps/app.manifest — but I don't see > it in the spec: http://mozilla.github.com/webapps-spec/ > > Is this in line with the spec? We probably need to get that up to date. Nick updated the dev docs here - https://developer.mozilla.org/en/Apps/Manifest
blocking-basecamp: --- → ?
blocking-kilimanjaro: --- → ?
Nom for k9o and basecamp - needed in order to allow listing of apps on marketplace that use preloading of app cache on web apps install. The corresponding bugs were blockers, so this should be a blocker as well. See bug 702369 and bug 760365.
OS: Mac OS X → All
Hardware: x86 → All
(In reply to Jason Smith [:jsmith] from comment #2) > > Is this in line with the spec? > > We probably need to get that up to date. Nick updated the dev docs here - > https://developer.mozilla.org/en/Apps/Manifest This appears twice in the example: "appcache_path": "/cache.manifest", ... "appcache_path": "/manifest.webapp"
(In reply to Chris Van Wiemeersch [:cvan] from comment #4) > (In reply to Jason Smith [:jsmith] from comment #2) > > > Is this in line with the spec? > > > > We probably need to get that up to date. Nick updated the dev docs here - > > https://developer.mozilla.org/en/Apps/Manifest > > This appears twice in the example: > > "appcache_path": "/cache.manifest", > ... > "appcache_path": "/manifest.webapp" Fixed. Only referencing /cache.manifest now.
The property is legit, I just tweaked the MDN docs a bit and updated the spec to include it.
Merged
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Nick: This isn't in production yet. It should get pushed next Thursday.
(In reply to Nick Desaulniers [:\n] from comment #9) > I'm still getting this error on: > https://marketplace-dev.allizom.org/en-US/developers/submit/app/manifest ??? Works on -dev with http://addax5313.testmanifest.com/manifest.webapp. Can you try again?
Working now!
Not an on-device bug. Removing nom.
blocking-basecamp: ? → ---
You need to log in before you can comment on or make changes to this bug.