Closed
Bug 821196
Opened 12 years ago
Closed 12 years ago
Installing a hosted app with locales present, but no default locale - installation should fail, but ends up being successful
Categories
(Core Graveyard :: DOM: Apps, defect)
Core Graveyard
DOM: Apps
Tracking
(blocking-basecamp:-)
RESOLVED
INVALID
blocking-basecamp | - |
People
(Reporter: jsmith, Unassigned)
References
()
Details
Steps:
1. Go to testmanifest.com
2. Install a web app that calls out support for locales, but fails to call out support for default locale
Expected:
Installation should fail in the core dom api saying the manifest is invalid.
Actual:
Installation ends up being successful.
Example:
{
"name":"Test App ({subdomain})",
"description":"This app has been automatically generated by testmanifest.com",
"version":"1.0",
"icons":{
"16":"http://testmanifest.com/icon-16.png",
"48":"http://testmanifest.com/icon-48.png",
"128":"http://testmanifest.com/icon-128.png"
},
"installs_allowed_from":[
"*"
],
"developer":{
"name":"Gregory Koberger",
"url":"http://gkoberger.net"
},
"locales":{
"es": {
"name": "Override name",
"developer": {
"name": "Override dev name",
"url": "Override url"
}
}
}
}
Reporter | ||
Updated•12 years ago
|
blocking-basecamp: --- → ?
Reporter | ||
Updated•12 years ago
|
Comment 1•12 years ago
|
||
Fabrice said that if there's no default locale we just pick the locale selected on the device. With that workaround and the fact that we're reviewing apps that go through the marketplace, let's not block on this.
blocking-basecamp: ? → -
Reporter | ||
Comment 2•12 years ago
|
||
(In reply to Andrew Overholt [:overholt] from comment #1)
> Fabrice said that if there's no default locale we just pick the locale
> selected on the device. With that workaround and the fact that we're
> reviewing apps that go through the marketplace, let's not block on this.
That assumption can no longer be made. Vivo is on the radar now. So we can't assume app review process can protect this anymore.
blocking-basecamp: - → ?
Let's update the docs since current behavior seems quite reasonable.
blocking-basecamp: ? → -
Reporter | ||
Comment 4•12 years ago
|
||
(In reply to Jonas Sicking (:sicking) from comment #3)
> Let's update the docs since current behavior seems quite reasonable.
Fine by me. Let's close this as invalid, but I'll put dev-doc-needed to update the spec.
Reporter | ||
Updated•12 years ago
|
No longer blocks: Apps-Dev-Doc-Needed
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•