Skip to content

Conversation

@rsickles
Copy link
Contributor

@rsickles rsickles commented Nov 4, 2025

Reformat stripe invoice note in billing overview

@rsickles rsickles requested a review from a team as a code owner November 4, 2025 20:00
@vercel
Copy link

vercel bot commented Nov 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
clickhouse-docs Ready Ready Preview Comment Nov 4, 2025 8:12pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
clickhouse-docs-jp Ignored Ignored Nov 4, 2025 8:12pm
clickhouse-docs-ru Ignored Ignored Nov 4, 2025 8:12pm
clickhouse-docs-zh Ignored Ignored Preview Nov 4, 2025 8:12pm

@dhtclk
Copy link
Collaborator

dhtclk commented Nov 4, 2025

@rsickles LGTM, one random point of discussion, on the billing index page it's a bit confusing clicking the billing overview link and getting taken to the marketplace billing overview page vs the actual billing overview page. I can make note of this and create an issue, but wanted to point that out. Let me know if I'm not making sense.

Copy link
Contributor Author

rsickles commented Nov 5, 2025

@dominictran not 100% sure - youre saying clicking this link takes you to the marketplace overview page instead? Im also working on a new billing overview page (linked here) that we will replace with the current page in the near future 🙂

@dhtclk
Copy link
Collaborator

dhtclk commented Nov 5, 2025

Screenshot 2025-11-04 at 7 56 39 PM

This link brings you to market place billing, which is fine, that's what it says. But I would expect there to be a link to the "pricing overview", or what's simply labeled as overview in the sidebar.

But not a huge deal especially if we're doing a new billing overview page anyway!

@rsickles
Copy link
Contributor Author

rsickles commented Nov 5, 2025

@dhtclk ohhh I see now - good catch! Just opened another PR for this change. Lets just make this small update now - thanks for finding it!

@Blargian Blargian merged commit 842ce88 into main Nov 5, 2025
15 checks passed
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.

4 participants