Replace LearnServiceMeshCTA with KanvasCTA across site #6569
Conversation
Signed-off-by: Ayush More <ayushmore42595@gmail.com>
Ayushmore1214
force-pushed
the
replace-learnservice-cta
branch
from
3384e93 to
e484d73
Compare
|
Can someone review my code and tell me what's wrong |
There was a problem hiding this comment.
No changes has to be made to this CTA. The ask is to simply replace its occurrence with any other CTA like KanvasCTA.
There was a problem hiding this comment.
@Ayushmore1214 I wonder if you have invested time to build your changes here? multiple broken reference to assets / components, resulting in failing build.
Please remove changes from package-lock and package.json files
Marking it as on/hold, please feel free to mark it as ready when it is.
|
@Ayushmore1214 Add it as an agenda item to the meeting minutes, if you would :) |
|
I'm working on this and will raise a PR soon |
Description
I have made changes asked accordingly in the issue
This PR fixes #6556
Notes for Reviewers
Signed commits