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

feat/config rate limit #1611

Merged
merged 50 commits into from
Oct 9, 2024
Merged

feat/config rate limit #1611

merged 50 commits into from
Oct 9, 2024

Conversation

harshithad0703
Copy link
Contributor

  • Rate limit implementation
  • @contentstack/cli-config: 1.7.1 -> 1.8.0

aman19K and others added 30 commits August 30, 2024 11:40
@aman19K
feat: added rate limit support in entries & assets bulk publish
cb7dcc1
@aman19K
formatting
eff430e
@aman19K
Merge pull request #1535 from contentstack/feat/DX-1238-bulk-publish-…
a82ea75 
…rate-limit

feat: added rate limit support in entries & assets bulk publish
@harshithad0703
implementation get and remove rate limit commands
d080f64
@harshithad0703
resolved the conversations, updated structure of rate limit, removed …
870c6fe 
…handleError
@harshithad0703
removed authentication for remove command
db2bdac
@harshithad0703
removed isAuthenticated from import
106c115
@harshithad0703
handle delete directly with config handler
01a4e13
@harshithad0703
formated get limit, removed message handler
c52b22e
@harshithad0703
Merge pull request #1543 from contentstack/feat/dx-1201-get-rate-limit
1b2cd56 
dx | 1201 | get and remove rate limit commands
@harshithad0703
added set rate limit command
f2384ce
@harshithad0703
Merge branch 'feat/config-rate-limit' into feat/DX-1112-rate-limit-se…
56b9cc9 
…t-command
@harshithad0703
_flags -> flag, remove console log, limit-name is optional, file rename
58eab0c
@harshithad0703
updates the limit name fetched from org plan
e615e30
@harshithad0703
remove console.log
e083e9b
@harshithad0703
update the limits passed through limit-name flag
e60ce66
@harshithad0703
added common utilities and other minor changes
c312864
@harshithad0703
added check for limit names, utilize values and comma separated value…
b9b0ff3 
…s for utilize
@aman19K
feat: set default rate limit using hook
3872fc3
@aman19K
formatting
83da4e5
@aman19K
Merge pull request #1551 from contentstack/feat/DX-1348
335df23 
feat: set default rate limit using hook
@harshithad0703
refactored code
3b1cc14
@harshithad0703
Merge pull request #1544 from contentstack/feat/DX-1112-rate-limit-se…
74072a0 
…t-command

dx | 1112 rate limit set command
@aman19K
Merge branch 'feat/config-rate-limit' into feat/bulk-publish-rate-limit
e3810b2
@aman19K
Merge pull request #1553 from contentstack/feat/bulk-publish-rate-limit
ce3eee1 
Feat/bulk publish rate limit
@harshithad0703
Merge branch 'development' into feat/config-rate-limit
5756494
@harshithad0703
added test cases for set rate limit command
5d96b83
@harshithad0703
added get and remove rate limit command test cases and updated set co…
31e0f8e 
…mmand
@harshithad0703
setup repo
5d09a81
@harshithad0703
added clius succes message after handling rate limit
4071f25
harshithad0703 and others added 15 commits September 13, 2024 14:47
@harshithad0703
improved coverage and updated readme
0512ee4
@harshithad0703
Merge pull request #1565 from contentstack/test/dx-1203-test-cases-ra…
486b743 
…te-limit-command

dx | 1203 test cases rate limit command
@harshithad0703
passing host in config and get limit value from org
802547d
@harshithad0703
Merge pull request #1571 from contentstack/fix/dx-1472-set-rate-limit…
f82132e 
…-org-issue

dx | 1472 set rate limit org issue
@harshithad0703
added abbreviation for config rate limits
5ea5fb6
@harshithad0703
Merge pull request #1574 from contentstack/feat/dx-1444-abbreviation-…
aa33566 
…config-rate-limit

dx | 1444 | abbreviation for config rate limits
@harshithad0703
Merge branch 'development' into fix/merge-conflicts
6feb2be
@harshithad0703
Merge pull request #1577 from contentstack/fix/merge-conflicts
8e50604 
merge conflicts (development and config rate limit
@harshithad0703
fix: update rate limits accordingly
8ab0c4b
@harshithad0703
Merge pull request #1605 from contentstack/fix/dx-1540-set-limits-acc…
5c8c6d0 
…ordingly

fix: update rate limits accordingly
@harshithad0703
version bump
0ad1796
@harshithad0703
Merge pull request #1608 from contentstack/fix/rate-limit-version-nump
fc65108 
version bump
@harshithad0703
Merge remote-tracking branch 'origin/development' into fix/rate-limit…
0496641 
…-merge-conflicts
@harshithad0703
update pnpm lock
ca9ecb9
@harshithad0703
Merge pull request #1610 from contentstack/fix/rate-limit-merge-confl…
fe34c46 
…icts

merge conflicts (development to feat/config-rate-limit)
@harshithad0703 harshithad0703 requested a review from a team as a code owner October 9, 2024 06:02
@harshithad0703 harshithad0703 marked this pull request as draft October 9, 2024 07:15
@harshithad0703 harshithad0703 marked this pull request as ready for review October 9, 2024 10:34
@harshithad0703 harshithad0703 merged commit f5b5c37 into development Oct 9, 2024
6 checks passed
cs-raj pushed a commit that referenced this pull request Nov 11, 2024
@harshithad0703
Merge pull request #1611 from contentstack/feat/config-rate-limit
6ddeb24 
feat/config rate limit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cs-raj cs-raj cs-raj approved these changes

@aman19K aman19K Awaiting requested review from aman19K

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 @cs-raj @aman19K