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

Cryptic error when creating a team with a whitespace via api #4827

Open
1 task done
kolaente opened this issue Aug 30, 2018 · 5 comments
Open
1 task done

Cryptic error when creating a team with a whitespace via api #4827

kolaente opened this issue Aug 30, 2018 · 5 comments
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented modifies/api This PR adds API routes or modifies them type/enhancement An improvement of existing functionality

Comments

@kolaente
Copy link
Member

Description

When creating a new team via the api which contains a whitespace, the api returns the following cryptic error:

[{"fieldNames":["Name"],"classification":"AlphaDashDotError","message":"AlphaDashDot"}]

After digging around in the code for two hours I finally found out the error was caused by a whitespace in the name.

I'd whish to get a more specific error message in that case, googling didn't help much either.

@lunny lunny added the type/enhancement An improvement of existing functionality label Aug 30, 2018
@stale
Copy link

stale bot commented Jan 11, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Jan 11, 2019
@lunny lunny added issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented and removed issue/stale labels Feb 7, 2019
@lunny
Copy link
Member

lunny commented Feb 7, 2019

Maybe added a not empty or required to the form tag?

@kolaente
Copy link
Member Author

kolaente commented Feb 7, 2019

@lunny sounds like a good idea

@6543 6543 added the modifies/api This PR adds API routes or modifies them label Sep 7, 2020
@davidgiga1993
Copy link

I'm facing the same issues. I would expect an API to give a proper error message why the validation failed. Otherwise one has to brute force it until it works..

@wxiaoguang
Copy link
Contributor

Isn't the message clear? AlphaDashDot means the field should contains alpha or dot or dash .....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented modifies/api This PR adds API routes or modifies them type/enhancement An improvement of existing functionality
Projects
None yet
Development

No branches or pull requests

5 participants