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

(OSS) Path Suffix Support for Rate Limit Quotas #15989

Merged
merged 9 commits into from
Jun 16, 2022
Merged

(OSS) Path Suffix Support for Rate Limit Quotas #15989

merged 9 commits into from
Jun 16, 2022

Conversation

VioletHynes
Copy link
Contributor

The OSS PR containing the OSS changes for path suffix support for rate limit quotas.

@VioletHynes VioletHynes marked this pull request as ready for review June 15, 2022 14:38
@VioletHynes VioletHynes requested review from ccapurso and hghaf099 June 15, 2022 17:20
ccapurso
ccapurso approved these changes Jun 16, 2022
View reviewed changes
Copy link
Contributor

@ccapurso ccapurso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left most of my comments on the ENT PR and it looks like you brought everything over :). Looks good!

@VioletHynes VioletHynes merged commit 3c95f6c into main Jun 16, 2022
@VioletHynes VioletHynes deleted the violethynes/path-suffix-quotas branch June 16, 2022 17:27
@VioletHynes VioletHynes restored the violethynes/path-suffix-quotas branch June 16, 2022 20:21
@VioletHynes VioletHynes deleted the violethynes/path-suffix-quotas branch June 16, 2022 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ccapurso ccapurso ccapurso approved these changes

@hghaf099 hghaf099 Awaiting requested review from hghaf099

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

Successfully merging this pull request may close these issues.
2 participants
@VioletHynes @ccapurso