Skip to content

feat: Add company billing page with subscription state overview#4541

Merged
Rockyy174 merged 2 commits into
operately:mainfrom
Rockyy174:subscription-overview-page
May 23, 2026
Merged

feat: Add company billing page with subscription state overview#4541
Rockyy174 merged 2 commits into
operately:mainfrom
Rockyy174:subscription-overview-page

Conversation

@Rockyy174

Copy link
Copy Markdown
Collaborator

No description provided.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've left some high level feedback:

  • In BillingStatusBadge, the className lookup assumes the BillingStatus union will never grow; consider adding a default/fallback style so new or unexpected statuses don’t render with missing styling.
  • The Current plan section shows the billing interval both under the plan heading and again in the detail rows; you may want to remove one of these to avoid redundant information.
  • A new paths.test.ts file is introduced but has no contents in this diff; if it’s not needed yet, consider removing it or populating it to avoid having a dead/placeholder test file in the tree.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- In `BillingStatusBadge`, the `className` lookup assumes the `BillingStatus` union will never grow; consider adding a default/fallback style so new or unexpected statuses don’t render with missing styling.
- The `Current plan` section shows the billing interval both under the plan heading and again in the detail rows; you may want to remove one of these to avoid redundant information.
- A new `paths.test.ts` file is introduced but has no contents in this diff; if it’s not needed yet, consider removing it or populating it to avoid having a dead/placeholder test file in the tree.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Rockyy174 added 2 commits May 23, 2026 14:52
Signed-off-by: Adriano Lazzaretti <lazzaretti136@gmail.com>
Signed-off-by: Adriano Lazzaretti <lazzaretti136@gmail.com>
@Rockyy174
Rockyy174 merged commit 30b6694 into operately:main May 23, 2026
3 checks passed
@Rockyy174
Rockyy174 deleted the subscription-overview-page branch May 23, 2026 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant