Curated group names can break group URLs

VERIFIED FIXED

Status

Participation Infrastructure
Phonebook
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: dpoirier, Assigned: dpoirier)

Tracking

2014-02.1

Details

(Whiteboard: [kb=1208141] [qa-])

(Assignee)

Description

4 years ago
On the curatedgroups feature branch, we're no longer creating groups through the groups field on the profile edit page, which was restricting group names to  a pretty minimal character set. On the new group creation form, we're currently allowing most chars in a group name. 

Some of the resulting slugs (the 'url' field on the group), when used to form URLs for the group app, result in complete URLs that don't match the patterns in the group app URL conf. 

For example, you can include ' in a group name (e.g. "Dan's Super Group") and the slug is something like "dan's_super_group", but ' is not matched by the patterns, so when you try to view the group, you get a 404.

We could add validation to the group form to restrict the group names back to what we had before, but that seems overly limiting. I think a better solution would be to allow pretty arbitrary group names, and just improve the code we use to generate the slug (url) so the slugs have the limited character set that'll match our URL patterns.
> I think a better
> solution would be to allow pretty arbitrary group names, and just improve
> the code we use to generate the slug (url) so the slugs have the limited
> character set that'll match our URL patterns.

+1 to this. Thanks!
(Assignee)

Comment 2

4 years ago
Pull request https://github.com/mozilla/mozillians/pull/741
Status: NEW → ASSIGNED
QA Contact: dpoirier
(Assignee)

Updated

4 years ago
Assignee: nobody → dpoirier
QA Contact: dpoirier
(Assignee)

Updated

4 years ago
Whiteboard: [kb=1208141]
Version: other → next
Marking verified as [qa-]. I suspect the unit tests coverage is sufficient. If there are manual test cases that I'm missing please unresolve the bug and I'll pounce on it.
Status: RESOLVED → VERIFIED
Whiteboard: [kb=1208141] → [kb=1208141] [qa-]
Version: next → 2014-02.1
You need to log in before you can comment on or make changes to this bug.