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

Wrong UI error when repo name contains underscore (on repo creation) #833

Closed
itaiad200 opened this issue Oct 18, 2020 · 6 comments · Fixed by #879
Closed

Wrong UI error when repo name contains underscore (on repo creation) #833

itaiad200 opened this issue Oct 18, 2020 · 6 comments · Fixed by #879
Assignees
Labels
area/UI Improvements or additions to UI bug Something isn't working hacktoberfest Issues that can get you awesome swag!

Comments

@itaiad200
Copy link
Contributor

No description provided.

@itaiad200 itaiad200 added area/UI Improvements or additions to UI bug Something isn't working hacktoberfest Issues that can get you awesome swag! labels Oct 18, 2020
@Sufiyan1997
Copy link
Contributor

@itaiad200 I would like to work on this.

@itaiad200
Copy link
Contributor Author

@Sufiyan1997 assigned. Feel free to reach out on slack if you need any help.

@Sufiyan1997
Copy link
Contributor

Do I need to change the regex to allow underscores in repository names or just change the error message?

@itaiad200
Copy link
Contributor Author

Underscores should not be allowed. We should validate the repo name in the UI and display a proper error message before sending the request to the server.

@Sufiyan1997
Copy link
Contributor

I have added client side validation for all 3 fields while creating repository.
Are this validation messages and UI correct?
Screenshot_2020-10-27 lakeFS App

@itaiad200
Copy link
Contributor Author

Almost :) It's easiest to point you to the validation on the server side to avoid mismatches.

  1. Swagger file:

    repository_creation:

  2. On the actual creation:

    if err := Validate(ValidateFields{

Let me know if something is unclear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/UI Improvements or additions to UI bug Something isn't working hacktoberfest Issues that can get you awesome swag!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants