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

🪟 Updates to credits page #15756

Merged
merged 5 commits into from
Aug 18, 2022
Merged

Conversation

natalyjazzviolin
Copy link
Contributor

@natalyjazzviolin natalyjazzviolin commented Aug 18, 2022

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:
Screen Shot 2022-08-18 at 10 14 52 AM

After:
Screen Shot 2022-08-18 at 10 16 12 AM

How

Removes the side menu component and conditional rendering for it. Adjusts bottom margin for the 'Remaining Credits' block.

Reading order

  1. CreditsPage.ts
  2. RemainingCredits.tsx

@natalyjazzviolin natalyjazzviolin requested a review from a team as a code owner August 18, 2022 14:19
@natalyjazzviolin natalyjazzviolin added area/platform issues related to the platform area/frontend Related to the Airbyte webapp labels Aug 18, 2022
Copy link
Contributor

@lmossman lmossman left a 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 />
Copy link
Contributor

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?

Copy link
Contributor Author

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!

@natalyjazzviolin natalyjazzviolin merged commit d52e55d into master Aug 18, 2022
@natalyjazzviolin natalyjazzviolin deleted the remove-menu-credits-page branch August 18, 2022 20:08
rodireich pushed a commit that referenced this pull request Aug 25, 2022
* 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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants