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)
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
Updated•13 years ago
|
Assignee: nobody → mattbasta
Comment 1•13 years ago
|
||
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?
Comment 2•13 years ago
|
||
(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
Updated•13 years ago
|
blocking-basecamp: --- → ?
blocking-kilimanjaro: --- → ?
Comment 3•13 years ago
|
||
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.
Updated•13 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
Comment 4•13 years ago
|
||
(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"
Comment 5•13 years ago
|
||
(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.
Comment 6•13 years ago
|
||
The property is legit, I just tweaked the MDN docs a bit and updated the spec to include it.
Assignee | ||
Comment 7•13 years ago
|
||
Assignee | ||
Comment 8•13 years ago
|
||
Merged
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 9•13 years ago
|
||
I'm still getting this error on: https://marketplace-dev.allizom.org/en-US/developers/submit/app/manifest ???
Assignee | ||
Comment 10•13 years ago
|
||
Nick: This isn't in production yet. It should get pushed next Thursday.
Comment 11•13 years ago
|
||
(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?
Reporter | ||
Comment 12•13 years ago
|
||
Working now!
You need to log in
before you can comment on or make changes to this bug.
Description
•