Bug #8406
Top-level menu nodes should allow empty paths
| Status: | Verified | Start date: | 05/07/2015 | |
|---|---|---|---|---|
| Priority: | Medium | Due date: | ||
| Assignee: | Nick Wilkinson | % Done: | 0% | |
| Category: | Form validation | |||
| Target version: | Release 2.5.0 | |||
| Google Code Legacy ID: | Tested version: | 2.2, 2.3 | ||
| Sponsored: | No | Requires documentation: |
Description
By default, our top-level menu nodes have the path field empty. If the user tries to edit a menu node, e.g. change the label of the staticPagesMenu node, the form won't validate unless the user introduces a path. A workaround is to type in a slash (/), but the validator should be tweaked in order to allow empty values for top-level nodes.
History
#1 Updated by Dan Gillean about 7 years ago
- Tested version 2.3 added
#2 Updated by Jesús García Crespo over 6 years ago
- Assignee deleted (
Jesús García Crespo)
#3 Updated by Mike Cantelon almost 4 years ago
- Status changed from New to Code Review
- Assignee set to Nick Wilkinson
PR for CR: https://github.com/artefactual/atom/pull/763
#4 Updated by Nick Wilkinson almost 4 years ago
- Assignee changed from Nick Wilkinson to Steve Breker
HI Steve, passing this to you for CR.
#5 Updated by Steve Breker almost 4 years ago
- Status changed from Code Review to Feedback
- Assignee changed from Steve Breker to Mike Cantelon
CR complete. Looks good Mike!
#6 Updated by Mike Cantelon almost 4 years ago
- Status changed from Feedback to QA/Review
- Assignee changed from Mike Cantelon to Dan Gillean
- Target version set to Release 2.5.0
Merged into qa/2.5.x for QA. Not sure if this is necessary to backport.
#7 Updated by Michelle Curran over 3 years ago
- Status changed from QA/Review to Verified
- Assignee changed from Dan Gillean to Nick Wilkinson
Verified in qa/2.5.x.