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.

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 plugin : change field of role maxLength from 10 to 50 #140

Merged
merged 1 commit into from Aug 14, 2021
Merged

fix plugin : change field of role maxLength from 10 to 50 #140

merged 1 commit into from Aug 14, 2021

Conversation

Switch-vov
Copy link
Contributor

No description provided.

@Switch-vov
Copy link
Contributor Author

I think the length of role should more than 10 character
image

@Switch-vov
Copy link
Contributor Author

@yu199195 After merge, should I update the submodule shenyu-dashboard in incubator-shenyu ?

@dengliming
Copy link
Member

@yu199195 After merge, should I update the submodule shenyu-dashboard in incubator-shenyu ?

@Switch-vov Yes.

@Switch-vov
Copy link
Contributor Author

@yu199195 After merge, should I update the submodule shenyu-dashboard in incubator-shenyu ?

@Switch-vov Yes.

Ok.

@dengliming dengliming merged commit 4665e8d into apache:master Aug 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dengliming dengliming dengliming approved these changes

@fengzhenbing fengzhenbing Awaiting requested review from fengzhenbing

Assignees
No one assigned
Labels
type: refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Switch-vov @dengliming