Skip to content

Replace LearnServiceMeshCTA with KanvasCTA #6556 #6671

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Rajesh-Nagarajan-11
Copy link
Contributor

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 commented Aug 1, 2025

Description

This PR fixes #6556
This PR updates the LearnServiceMeshCTA component to KanvasCTA across the following files:

1.subscribe.js
2.unsubscribe.js
3.unsubscribed.js
4.service-mesh-book.js
5.thank-you.js

All instances of LearnServiceMeshCTA have been replaced to align with the latest call-to-action direction.

Notes for Reviewers
Please check the preview pages to ensure the KanvasCTA is rendering properly:

1./thank-you

Screenshot from 2025-08-05 08-10-44

2./subscribe
Screenshot from 2025-08-04 19-25-50

3./unsubscribe

Screenshot from 2025-08-04 19-26-16

4./unsubscribed
Screenshot from 2025-08-05 08-10-39

5./learn/service-mesh-books
Screenshot from 2025-08-04 19-26-40

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Rajesh-Nagarajan-11 <[email protected]>
@l5io
Copy link
Contributor

l5io commented Aug 1, 2025

🚀 Preview for commit c869939 at: https://688c85e8202cdcae5559678e--layer5.netlify.app

@vr-varad
Copy link
Contributor

vr-varad commented Aug 4, 2025

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

@leecalcote
Copy link
Member

@Rajesh-Nagarajan-11 offer a screenshot or hyperlink to a preview page with this new CTA.

@Rajesh-Nagarajan-11
Copy link
Contributor Author

@Rajesh-Nagarajan-11 offer a screenshot or hyperlink to a preview page with this new CTA.

@leecalcote Updated in PR Description

@avni-mahajan
Copy link

avni-mahajan commented Aug 4, 2025

Hi @Rajesh-Nagarajan-11,

Thank you for contributing! LGTM!

I noticed that one other file: src/templates/event-single.js still references LearnServiceMeshCTA.

Could you update that one to KanvasCTA as well?

I also noticed a couple of design inconsistencies:

  • On the thank-you page, the “Kanvas” text is green which should be white

  • On the unsubscribed page, the “Kanvas” text is not bold

Would you be able to fix these as well for consistency?

@Rajesh-Nagarajan-11
Copy link
Contributor Author

Rajesh-Nagarajan-11 commented Aug 5, 2025

@avni-mahajan Yeah thank you avni !! Event-single.js page already has another kanvas CTA checkout that and let me check with design inconsistency 👍🏻

Event-single.js
image

Signed-off-by: Rajesh-Nagarajan-11 <[email protected]>
@Rajesh-Nagarajan-11
Copy link
Contributor Author

Rajesh-Nagarajan-11 commented Aug 5, 2025

@avni-mahajan Updated everthing what u suggested , Kindly review once ! thank Youu !!

@l5io
Copy link
Contributor

l5io commented Aug 5, 2025

🚀 Preview for commit 4c98417 at: https://689172923493b6d65358696a--layer5.netlify.app

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.

Update Call To Action
5 participants