Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[website] Add blog link to # table #43123

Merged
merged 1 commit into from
Aug 1, 2024
Merged

[website] Add blog link to # table #43123

merged 1 commit into from
Aug 1, 2024

Conversation

zanivan
Copy link
Contributor

@zanivan zanivan commented Jul 30, 2024

This PR is to add information about the 10 seat cap removal and early bird # after August.

I added some tweaks to the text, and added a link to the blog post (TBD).

👉 https://deploy-preview-43123--material-ui.netlify.app/#/

@zanivan zanivan added the website Pages that are not documentation-related, marketing-focused. label Jul 30, 2024
@zanivan zanivan self-assigned this Jul 30, 2024
No extra fees for orders with over 10 devs 
<span>
<Tooltip title="Our # policies are changing. Read more on our blog.">
<Link href="https://mui.com/#">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@joserodolfofreitas let me know which link we should have here, I'm using this as a placeholder for now

@zanivan zanivan added the needs cherry-pick The PR should be cherry-picked to master after merge label Jul 30, 2024
Copy link
Member

@joserodolfofreitas joserodolfofreitas left a comment

Choose a reason for hiding this comment

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

Thank you, @zanivan!
I've added the links as comments on the PR.

No extra fees for orders with over 10 devs&nbsp;
<span>
<Tooltip title="Our # policies are changing. Read more on our blog.">
<Link href="https://mui.com/#">
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Link href="https://mui.com/#">
<Link href="/blog/mui-x-sep-2024-price-update/">

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed in #43141

🐦 Early Bird: <strong>25% off</strong> if ordered &nbsp;
<span>
<Tooltip title="Our # policies are changing. Read more on our blog.">
<Link href="https://mui.com/#">
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Link href="https://mui.com/#">
<Link href="/blog/mui-x-sep-2024-price-update/">

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed in #43141

@JCQuintas JCQuintas merged commit b33fca9 into next Aug 1, 2024
27 checks passed
@JCQuintas JCQuintas deleted the #-table-changes branch August 1, 2024 10:26
github-actions bot pushed a commit that referenced this pull request Aug 1, 2024
zanivan added a commit that referenced this pull request Aug 1, 2024
@zanivan
Copy link
Contributor Author

zanivan commented Aug 1, 2024

@JCQuintas I believe this was merged without updating the links in the # table— @joserodolfofreitas I'll open a new PR to fix it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
needs cherry-pick The PR should be cherry-picked to master after merge website Pages that are not documentation-related, marketing-focused.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants