Skip to content

feat: improvement validate the information input from the user for various sources#205

Merged
iziang merged 0 commit intomainfrom
152-improvement-validate-the-information-input-from-the-user-for-various-sources
Sep 25, 2023
Merged

feat: improvement validate the information input from the user for various sources#205
iziang merged 0 commit intomainfrom
152-improvement-validate-the-information-input-from-the-user-for-various-sources

Conversation

@yipeng1030
Copy link
Contributor

add an error handler when creating collections and config is wrong.

@apecloud-bot apecloud-bot added the size/M Denotes a PR that changes 30-99 lines. label Sep 12, 2023
@apecloud-bot
Copy link
Collaborator

This branch name is not following the standards: feature/|bugfix/|release/|hotfix/|support/|releasing/|dependabot/

@iziang
Copy link
Contributor

iziang commented Sep 12, 2023

It would be better to add a validation API that can be called before creating a collection to validate the input information for its legality.

@yipeng1030 yipeng1030 changed the title improvement validate the information input from the user for various sources feat: improvement validate the information input from the user for various sources Sep 13, 2023
@apecloud-bot
Copy link
Collaborator

This branch name is not following the standards: feature/|bugfix/|release/|hotfix/|support/|releasing/|dependabot/

@iziang iziang marked this pull request as ready for review September 25, 2023 10:04
@apecloud-bot apecloud-bot added size/L Denotes a PR that changes 100-499 lines. and removed size/M Denotes a PR that changes 30-99 lines. labels Sep 25, 2023
@iziang iziang merged this pull request into main Sep 25, 2023
@iziang iziang deleted the 152-improvement-validate-the-information-input-from-the-user-for-various-sources branch September 25, 2023 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L Denotes a PR that changes 100-499 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement] validate the information input from the user for various sources.

3 participants