If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

"`default_locale` without `locales`" should be "`locales` without `default_locale`"

RESOLVED FIXED in 2012-08-02

Status

Marketplace
Validation
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: eviljeff, Assigned: cvan)

Tracking

2012-08-02
Points:
---

Details

(Reporter)

Description

5 years ago
The validation rule 
`default_locale` without `locales` 
is matching the wrong way around - it should be okay to have a default locale and no other locales, but its not okay to have locales listed and no default.
(Assignee)

Comment 1

5 years ago
Yep, this is what the spec says: http://mozilla.github.com/webapps-spec/#properties
(Reporter)

Comment 2

5 years ago
Any chance we can get this fixed before the next push?  Its filling up the re-review queue with false reviews.
Target Milestone: --- → 2012-08-02
(Assignee)

Updated

5 years ago
Assignee: nobody → cvan
(Reporter)

Comment 3

5 years ago
https://github.com/mozilla/amo-validator/pull/176
(Assignee)

Comment 4

5 years ago
https://github.com/mozilla/amo-validator/commit/289e398

Thanks, Andrew!
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.