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: updates the VeriryPermission API to make the objectName parameter optional instead of required #255

Merged
merged 1 commit into from
May 22, 2023
Merged

fix: updates the VeriryPermission API to make the objectName parameter optional instead of required #255

merged 1 commit into from
May 22, 2023

Conversation

fynnss
Copy link
Contributor

@fynnss fynnss commented May 19, 2023 •
edited
Loading

Description

Updates the VeriryPermission API to make the objectName parameter optional instead of required

Rationale

For verifying bucket permissions, the objectName is no required.

Example

image

Changes

Notable changes:

  • NA

@fynnss fynnss changed the title fix: verify permission openapi params fix: updates the VeriryPermission API to make the objectName parameter optional instead of required May 19, 2023
@fynnss fynnss requested review from drjackchan and unclezoro May 19, 2023 12:31
@fynnss fynnss added the r4r label May 22, 2023
@unclezoro unclezoro merged commit 9fa39d9 into develop May 22, 2023
@unclezoro unclezoro mentioned this pull request May 23, 2023
Merged
forcodedancing pushed a commit that referenced this pull request May 25, 2023
@fynnss @forcodedancing
fix: verify permission openapi params (#255)
226b036
@fynnss fynnss deleted the fix_verify_permission_params branch July 21, 2023 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jingjunLi jingjunLi jingjunLi approved these changes

@unclezoro unclezoro unclezoro approved these changes

@drjackchan drjackchan Awaiting requested review from drjackchan

Assignees
No one assigned
Labels
r4r
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fynnss @unclezoro @jingjunLi