-
Notifications
You must be signed in to change notification settings - Fork 115
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
Conversation
…blic Design Signed-off-by: Vidit Kushwaha <viditkushwaha530@gmail.com>
There was a problem hiding this 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.
yes, I have checked it will change behaviour of catalog page |
that's what we don't want |
we want published for the catalog page and public for design page |
Is it done? |
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 |
I think we already passed the props that necessary to differentiate the catalog and my-designs page |
Signed-off-by: Vidit Kushwaha <viditkushwaha530@gmail.com>
Yes we do pass relevant props, pushed changeous to PR Can you review again, @amitamrutiya |
Signed-off-by: Vidit Kushwaha <viditkushwaha530@gmail.com>
@amitamrutiya pushed changeous |
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.
Signed commits