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.

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

Feat/jovu usage limits #8358

Merged
merged 5 commits into from May 1, 2024
Merged

Conversation

yuval-hazaz
Copy link
Member

Close: https://github.com/amplication/private-issues/issues/183

PR Details

Add enforcement for the number of Jovu requests per a new entitlement.
image

PR Checklist

  • Tests for the changes have been added
  • npm test doesn't throw any error

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@yuval-hazaz
Copy link
Member Author

@PazYanoverr @mulygottlieb please note we need to add the new entitlement in the prod env
https://app.stigg.io/development-0239228d/features/feature-jovu-requests

Copy link

nx-cloud bot commented Apr 30, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit dda7577. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

Base automatically changed from feat/jovu-create-action to feat/jovu-get-actions May 1, 2024 08:33
Base automatically changed from feat/jovu-get-actions to feat/jovu-create-enum May 1, 2024 08:34
Base automatically changed from feat/jovu-create-enum to feat/jovu-api May 1, 2024 08:34
Base automatically changed from feat/jovu-api to feat/jovu-plugins May 1, 2024 08:35
@yuval-hazaz yuval-hazaz merged commit 9484883 into feat/jovu-plugins May 1, 2024
20 of 21 checks passed
@yuval-hazaz yuval-hazaz deleted the feat/jovu-usage-limits branch May 1, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants