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

Fix .access(...) parameter #12676

Closed

Conversation

el-hopaness-romtic
Copy link
Contributor

This commit fixed first example but not the second one (.access(...) parameter's class is AuthorizationManager, not String)

@pivotal-cla
Copy link

@el-hopaness-romtic Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 16, 2023
@el-hopaness-romtic
Copy link
Contributor Author

@pivotal-cla This is an Obvious Fix

1 similar comment
@jzheaux
Copy link
Contributor

jzheaux commented Mar 20, 2023

@pivotal-cla This is an Obvious Fix

@jzheaux jzheaux self-assigned this Mar 20, 2023
@pivotal-cla
Copy link

@el-hopaness-romtic This Pull Request contains an obvious fix. Signing the Contributor License Agreement is not necessary.

@jzheaux jzheaux added in: docs An issue in Documentation or samples type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 20, 2023
@jzheaux jzheaux added this to the 6.1.0-M3 milestone Mar 20, 2023
@jzheaux
Copy link
Contributor

jzheaux commented Mar 20, 2023

Thanks, @el-hopaness-romtic! This is now merged into 5.8.x, 6.0.x, and main. I hope to see more contributions from you in the future.

@jzheaux jzheaux closed this Mar 20, 2023
@jzheaux jzheaux modified the milestones: 6.1.0-M3, 5.8.3 Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: docs An issue in Documentation or samples type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants