- **status**: review --> in-progress
- **Comment**:
* the slugify function allows periods, is that ok? It's different from what
trove shortnames currently allow, but it probably is ok? A few test cases for
it should be added, and its docstring should be updated.
* `AddTroveCategoryForm.display()` has a few commented out lines
* I'd probably put the Name column before Shortname column, in the listings
table, but that's a small personal preference.
* adding a category with unicode in the name gets an error on
`redirect('/categories/{}/?categoryname={}&shortname={}'.format(upper.trove_cat_id,
name, shortname))` When using format strings `u'...'` is needed. It is
unfortunately not magical like `%s` strings which upgrade to unicode
automatically when the params are unicode.
* new top-level troves no longer cause the project admin page to break
immediately, but they do show up there and can't be saved. Might need a config
option to control which categories are used there. Can go alongside
`trovecategories.admin.*` ini settings.
---
** [tickets:#8182] Improve category management screens**
**Status:** in-progress
**Milestone:** unreleased
**Created:** Thu Feb 01, 2018 02:48 PM UTC by Kenton Taylor
**Last Updated:** Thu Feb 01, 2018 03:55 PM UTC
**Owner:** Kenton Taylor
Adding a new top-level trove via the admin UI causes Project
Admin->Categorization to break.
The current admin also doesn't allow identical names in another (unrelated)
trove -- add a field to override the auto-generated shortname, and improve
shortname generation to allow for more friendly URLs.
---
Sent from forge-allura.apache.org because [email protected] is subscribed
to https://forge-allura.apache.org/p/allura/tickets/
To unsubscribe from further messages, a project admin can change settings at
https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.
