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

dx | 1201 | get and remove rate limit commands #1543

Merged
merged 6 commits into from
Sep 4, 2024
Merged

dx | 1201 | get and remove rate limit commands #1543

merged 6 commits into from
Sep 4, 2024

Conversation

harshithad0703
Copy link
Contributor

  • get rate limit command
  • remove rate limit command
  • version bump to be made

@harshithad0703 harshithad0703 changed the base branch from development to feat/config-rate-limit September 3, 2024 11:22
@harshithad0703 harshithad0703 changed the title dx | 1201 | implementation of rate limit commands dx | 1201 | get and reomve rate limit commands Sep 3, 2024
@harshithad0703 harshithad0703 changed the title dx | 1201 | get and reomve rate limit commands dx | 1201 | get and remove rate limit commands Sep 3, 2024
@harshithad0703 harshithad0703 marked this pull request as ready for review September 3, 2024 11:25
@harshithad0703 harshithad0703 requested a review from a team as a code owner September 3, 2024 11:25
aman19K
aman19K requested changes Sep 3, 2024
View reviewed changes
packages/contentstack-config/src/commands/config/get/rate-limit.ts Show resolved Hide resolved
packages/contentstack-config/src/commands/config/get/rate-limit.ts Outdated Show resolved Hide resolved
packages/contentstack-config/src/commands/config/get/rate-limit.ts Outdated Show resolved Hide resolved
packages/contentstack-config/src/commands/config/get/rate-limit.ts Outdated Show resolved Hide resolved
packages/contentstack-config/src/commands/config/get/rate-limit.ts Outdated Show resolved Hide resolved
packages/contentstack-config/src/commands/config/remove/rate-limit.ts Outdated Show resolved Hide resolved
packages/contentstack-config/src/commands/config/remove/rate-limit.ts Outdated Show resolved Hide resolved
packages/contentstack-config/src/commands/config/remove/rate-limit.ts Outdated Show resolved Hide resolved
packages/contentstack-config/src/commands/config/remove/rate-limit.ts Show resolved Hide resolved
packages/contentstack-config/src/utils/interactive.ts Outdated Show resolved Hide resolved
@harshithad0703 harshithad0703 marked this pull request as draft September 4, 2024 04:50
@harshithad0703 harshithad0703 marked this pull request as ready for review September 4, 2024 06:42
@harshithad0703 harshithad0703 merged commit 4534bbc into feat/config-rate-limit Sep 4, 2024
4 checks passed
@harshithad0703 harshithad0703 deleted the feat/dx-1201-get-rate-limit branch September 4, 2024 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shafeeqfalcon shafeeqfalcon shafeeqfalcon approved these changes

@aman19K aman19K aman19K approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@harshithad0703 @shafeeqfalcon @aman19K