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.

Sign up for GitHub

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

Jump to bottom

[Fix] fix data-permission namespace problem #5654

Merged

xcsnx merged 2 commits into apache:master from VampireAchao:fix_data_permission_namespace

Sep 9, 2024

Contributor

VampireAchao commented Sep 9, 2024

"Use @Length(max = to validate the length of a String instead of @Max(value = because @Max only verifies numbers."
"Do not query dataPermission when selectorIds is empty."

Make sure that:

You have read the contribution guidelines.
You submit test cases (unit or integration tests) that back your changes.
Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.


          [Fix] fix data-permission namespace problem

e54a2a4

VampireAchao requested a review from xcsnx

September 9, 2024 09:17

VampireAchao self-assigned this


          [style] fix code style for ci follow checkstyle

abcfe27

xcsnx added the namespace label

xcsnx approved these changes

View reviewed changes

xcsnx merged commit cc2df49 into apache:master

45 checks passed

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

Labels