Fix: enforce minimum team name length (>=3) in UI#5292
Conversation
DerDreschner
left a comment
There was a problem hiding this comment.
Hey, thank you very much for your pull request! Could you please change the code so the CI runs are green?
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
2a286cc to
ec5fb91
Compare
Signed-off-by: JSchut96 <jeroen.schut@hotmail.com>
ec5fb91 to
3a8a0b5
Compare
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! (If you believe you should not receive this message, you can add yourself to the blocklist.) |
Issue
Based on: #3254
The backend enforces a minimum team name length of 3 characters and returns a 400 Bad Request when this requirement is not met (e.g. "Circle name is too short").
However, the UI does not properly handle or display the error message from the server, leaving the user without a clear indication of why their submission failed.
Proposed Solution
Instead of focusing on improving error message display, this fix prevents invalid team names from being submitted altogether.
The solution: