-
Notifications
You must be signed in to change notification settings - Fork 107
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
Updating the Help Links of all the components #206
Conversation
✅ Deploy Preview for circuitverse ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Hi @Prerna-0202 @Arnabdaz , |
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.
Please change the link in Stepper.js, Splitter.js and Demultiplexer.js also.
Yes @Prerna-0202 , I made those changes! |
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.
Great 🎉
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.
Fixes #206
Describe the changes you have made in this PR -
Updated all the Help Links in the required files (except the ones in which there is no docs regarding that page)
Screenshots of the changes (If any) -
https://circuitverse.netlify.app/simulatorvue
When we click on the help button of any component then it redirect us to the PageNotFound Page (Because the links were not updated)
PageNotFound
But after changing the links to the correct ones it works !
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.