Skip to content
This repository has been archived by the owner on Feb 11, 2024. It is now read-only.

Panic when deploying after kick merge #113

Closed
Brawdunoir opened this issue Aug 28, 2022 · 0 comments · Fixed by #114
Closed

Panic when deploying after kick merge #113

Brawdunoir opened this issue Aug 28, 2022 · 0 comments · Fixed by #114
Assignees
Labels
bug Something isn't working

Comments

@Brawdunoir
Copy link
Collaborator

#111 results in a panic in render during deployment.

panic: ':roomid' in new path '/rooms/:roomid/kick/:userid' conflicts with existing wildcard ':id' in existing prefix '/rooms/:id'

Apparently, we just need to rename :roomid to :id in route.

Basically, both routes need to have the wildcard in the same position, and the wildcards need to be named the same thing.

@Brawdunoir Brawdunoir added the bug Something isn't working label Aug 28, 2022
@Brawdunoir Brawdunoir self-assigned this Aug 28, 2022
@Brawdunoir Brawdunoir linked a pull request Aug 28, 2022 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant