Closed Bug 911781 Opened 6 years ago Closed 6 years ago

Implement a module to validate the app manifest

Categories

(DevTools :: WebIDE, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 26

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(2 files, 3 obsolete files)

We need a module to help the developer figuring out various kind of error that can happen around its app manifest like bad JSON, wrong property values or missing files...
Attachment #798558 - Flags: review?(paul)
Comment on attachment 798558 [details] [diff] [review]
Implement a module to validate the app manifest

Let's not have errors messages hidden in the code.

Can you use a .properties file. I'm not saying we should get them localized, but I'd rather have these messages all listed in one file.
Attachment #798558 - Flags: review?(paul) → review-
Blocks: 912447
Blocks: appmgr_v1
Attachment #798558 - Attachment is obsolete: true
Attachment #800062 - Attachment is obsolete: true
Attachment #800068 - Flags: review?(paul)
Attachment #800068 - Flags: review?(paul) → review+
Comment on attachment 800112 [details] [diff] [review]
Implement a module to validate the app manifest

See previous patch for interdiff.
Attachment #800112 - Flags: review?(paul)
Attachment #800112 - Flags: review?(paul) → review+
https://hg.mozilla.org/mozilla-central/rev/b92cf59aa9d7
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Comment on attachment 800112 [details] [diff] [review]
Implement a module to validate the app manifest

Review of attachment 800112 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/locales/en-US/chrome/browser/devtools/app-manager.properties
@@ +12,5 @@
> +validator.wrongManifestFileName=Packaged apps require a manifest file that can only be named 'manifest.webapp' at project root folder
> +validator.invalidManifestURL=Invalid manifest URL '%S'
> +validator.invalidManifestJSON=The webapp manifest isn't a valid JSON file: %1$S at: %2$S
> +validator.noAccessManifestURL=Unable to read manifest file: %1$S at: %2$S
> +validator.invalidHostedManifestURL=Invalid hosted manifest URL '%1$S': %2$S

l10n comments would definitely help for these 3 strings.

@@ +16,5 @@
> +validator.invalidHostedManifestURL=Invalid hosted manifest URL '%1$S': %2$S
> +validator.invalidProjectType=Unknown project type '%S'
> +validator.missNameManifestProperty=Missing mandatory 'name' in Manifest.
> +validator.missIconsManifestProperty=Missing 'icons' in Manifest.
> +validator.missIconForMarketplace=app submission to the Marketplace needs at least an 128 icon

Is "an 128 icon" correct in English?
Depends on: 913366
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.