-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
🪟 Updates to credits page #15756
🪟 Updates to credits page #15756
Conversation
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.
Had one small nit about component naming but otherwise this looks good to me
</Suspense> | ||
</MainView> | ||
</MainInfo> | ||
<CreditsUsagePage /> |
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.
nit: should this component be renamed to something like CreditsUsage
, since it is no longer being considered as a "separate page", but instead just a part of the CreditsPage?
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.
@lmossman That's a really good point, thank you!
* Removes side menu. * Removes conditional rendering that is no longer needed. * Adds bottom margin to the Remaining Credits block. * Renames credit usage component as per request.
What
Closes https://github.com/airbytehq/airbyte-cloud/issues/2308
Part of https://github.com/airbytehq/airbyte-cloud/issues/2130
Removes the 'Credit Usage' side menu on the /credits page.
Before:

After:

How
Removes the side menu component and conditional rendering for it. Adjusts bottom margin for the 'Remaining Credits' block.
Reading order
CreditsPage.ts
RemainingCredits.tsx