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

External API authorization allow empty value for pagination params #6894

Merged
merged 2 commits into from
Oct 29, 2023
Merged

External API authorization allow empty value for pagination params #6894

merged 2 commits into from
Oct 29, 2023

Conversation

nopcoder
Copy link
Contributor

No description provided.

@nopcoder nopcoder added the exclude-changelog PR description should not be included in next release changelog label Oct 28, 2023
@nopcoder nopcoder self-assigned this Oct 28, 2023
@github-actions
Copy link

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

1 similar comment
@github-actions
Copy link

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

@nopcoder nopcoder requested review from guy-har and N-o-Z October 29, 2023 10:18
@nopcoder nopcoder added the minor-change Used for PRs that don't require issue attached label Oct 29, 2023
@nopcoder nopcoder merged commit c490210 into master Oct 29, 2023
31 of 32 checks passed
@nopcoder nopcoder deleted the fix/auth-allow-empty branch October 29, 2023 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guy-har guy-har guy-har approved these changes

@N-o-Z N-o-Z Awaiting requested review from N-o-Z

Assignees

@nopcoder nopcoder

Labels
exclude-changelog PR description should not be included in next release changelog minor-change Used for PRs that don't require issue attached
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nopcoder @guy-har