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

Fixed router url #157 #165

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

Conversation

vansh-codes
Copy link

@vansh-codes vansh-codes commented Jun 26, 2024

Fixes #157

Fixed the URL of all pages

Earlier

image
image

After PR MERGE

image
image

Description about changes

The unnecessary '#' symbol from the URl's have been removed by changing the router model to 'BrowserRouter'.
Added tags in place of (anchor) tag wherever necessary (mainly in footer) for faster navigation through routes and saving the user from unnecassary reloading(refreshing) time.

@vansh-codes
Copy link
Author

@adarsh-singh01 Please review my PR. I have fixed the routing of the website.

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.

Fix url routing for all pages
1 participant