-
Notifications
You must be signed in to change notification settings - Fork 389
new: STORIF-183 - Quotas table modified to show throughputs. #13177
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
base: develop
Are you sure you want to change the base?
Conversation
1aa8634 to
d6e02aa
Compare
akalcosmin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The decision model for showing or hiding quotas will change in the final design(display mode is replaced with a quota_type_id hard-coded list). Until this part is implemented, we should not merge this PR.
e51b45c to
c0b4836
Compare
|
@akalcosmin Hello, thank you for you comment. I have updated the approach, we should be able to merge it after all the related apinext PRs are merged . |
02494bc to
99b32e8
Compare
99b32e8 to
17db5d9
Compare
17db5d9 to
c069040
Compare
Cloud Manager UI test results🎉 867 passing tests on test run #16 ↗︎
|
Description 📝
Quotas table modified to show throughputs.
Preview 📷
How to test 🧪
docker compose up -d apinext.envfile inside the cloud manager to use local instance of the apinext.pnpm dev./quotas.Related PRs
Author Checklists
As an Author, to speed up the review process, I considered 🤔
👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support
As an Author, before moving this PR from Draft to Open, I confirmed ✅