Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error message from course slug collision does not explain the problem #6087

Open
ragesoss opened this issue Jan 3, 2025 · 3 comments · May be fixed by #6168
Open

Error message from course slug collision does not explain the problem #6087

ragesoss opened this issue Jan 3, 2025 · 3 comments · May be fixed by #6168

Comments

@ragesoss
Copy link
Member

ragesoss commented Jan 3, 2025

What is happening?

Attempting to change a course slug component (such as title) will result in a server error if it changes the slug to match another course. However, the error message shown in the UI is just "Internal Server Error". It should instead explain the problem so that the user can choose a different string, or navigate to the other course and delete it (if it's not being used).

@AliyanA1
Copy link

AliyanA1 commented Jan 4, 2025

Hi maintainer, 👋

This is my first pull request, and I’m excited to contribute! 🎉
I’m new to open source, so I’d greatly appreciate any feedback on my changes.

Please let me know if there’s anything I need to improve or if I missed something important. I’m happy to learn and make adjustments. Thank you for reviewing my PR!

Best regards,
AliyanA1

@Jay2006sawant
Copy link
Contributor

Jay2006sawant commented Jan 15, 2025

@ragesoss I would like to take on this task and work towards resolving it.

@devrajpardhi04
Copy link

Can you please upload a video

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment