Group name should be case insensitive in /my route

Bug
1

Continuing the discussion from Make "/my" work for all possible links to current user:

I was really confused why this didn’t work for me, as this was what I tried first when I read that topic.

After some testing I think I found the cause: As soon as there is a capital letter within the group name this fails.

My steps are always the same:

  1. I visit my inbox
  2. I use the dropdown to go to the inbox of the group
  3. I replace /u/Moin with /my
  4. I check if this works

Verifying this works without a capital letter:
Group name: group_name
Inbox path: /u/Moin/messages/group/group_name
Replaced: /my/messages/group/group_name
→ :check_mark: replacing with my works

Now testing with a capital letter:
Group name: Group_name
Inbox path: /u/Moin/messages/group/Group_name
Replaced: /my/messages/group/Group_name
→ :cross_mark: replacing with my doesn’t work
It works when I use /my/messages/group/group_name

Group name: group_Name
Inbox path: /u/Moin/messages/group/group_Name
Replaced: /my/messages/group/group_Name
→ :cross_mark: replacing with my doesn’t work
It works when I use /my/messages/group/group_name

How am I supposed to know that I also need to convert the name to lowercase?

3 Likes
2

No, I think this should probably be case insensitive. Plucking this out to a new topic (Bug or UX? either one works for me)

1 Like
3

Reminds me of Route capitalization error in private_messages_group - #6