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

fix(my-design): Update heading for Other Published Design to Other Public Design #888

Merged
merged 5 commits into from
Feb 2, 2025

Conversation

Vidit-Kushwaha
Copy link
Contributor

Notes for Reviewers

This PR updates the heading on the "My Design" page from "Other Published Design" to "Other Public Design" for better clarity and consistency with terminology.

image

Signed commits

  • Yes, I signed my commits.

…blic Design

Signed-off-by: Vidit Kushwaha <viditkushwaha530@gmail.com>
Copy link
Contributor

@amitamrutiya amitamrutiya left a comment

Choose a reason for hiding this comment

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

Check is will going to change the behavior for the catalog page as well.

@Vidit-Kushwaha
Copy link
Contributor Author

Check is will going to change the behavior for the catalog page as well.

yes, I have checked it will change behaviour of catalog page

@amitamrutiya
Copy link
Contributor

yes, I have checked it will change behaviour of catalog page

that's what we don't want

@amitamrutiya
Copy link
Contributor

we want published for the catalog page and public for design page

@amitamrutiya
Copy link
Contributor

Is it done?

@leecalcote
Copy link
Member

@Vidit-Kushwaha ?

@Vidit-Kushwaha
Copy link
Contributor Author

To have a different text for different pages (catalog page and design page) we can add one more prop to component to check whether page is catalog or not or we can do pass a heading as an prop

@amitamrutiya
Copy link
Contributor

I think we already passed the props that necessary to differentiate the catalog and my-designs page

@Vidit-Kushwaha
Copy link
Contributor Author

Vidit-Kushwaha commented Feb 1, 2025

Yes we do pass relevant props, pushed changeous to PR Can you review again, @amitamrutiya

Signed-off-by: Vidit Kushwaha <viditkushwaha530@gmail.com>
@Vidit-Kushwaha
Copy link
Contributor Author

@amitamrutiya pushed changeous

@amitamrutiya amitamrutiya merged commit 89ebe64 into layer5io:master Feb 2, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants