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

Created Login Page and Update Navigation. #82

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

Conversation

Divyanjali-Mudili
Copy link
Contributor

issue : #7

Description:

  • Created the login page for the website.
  • Edited the "Sign In" button for better user experience.
  • Added navigation links to index.html, about.html, contact.html, features.html, and reviews.html.

Copy link

vercel bot commented Jan 29, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fusion-vision ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 1, 2025 6:07pm

@Divyanjali-Mudili Divyanjali-Mudili changed the title Create Login Page and Update Navigation (#7) Create Login Page and Update Navigation. Jan 29, 2025
@Divyanjali-Mudili Divyanjali-Mudili changed the title Create Login Page and Update Navigation. Created Login Page and Update Navigation. Jan 29, 2025
@sudiptasarkar011
Copy link
Owner

I can see the file changes but cannot see the changes in the deployed version of this PR. Moreover when I am clicking the sign in or get started button it is showing error.

Also, I appreciate your efforts and if you want to work on this issue its fine but make sure to get it assigned first.

@sudiptasarkar011
Copy link
Owner

@Divyanjali-Mudili are you looking into this or shall I close the PR?

@Divyanjali-Mudili
Copy link
Contributor Author

@sudiptasarkar011 yes I'm looking into the problem.

@sudiptasarkar011
Copy link
Owner

@Divyanjali-Mudili any update?

@Divyanjali-Mudili
Copy link
Contributor Author

Resolved the issue. Should I again raise the pr?

@sudiptasarkar011
Copy link
Owner

Yes sure go ahead

@Divyanjali-Mudili
Copy link
Contributor Author

@sudiptasarkar011 please review my pr and merge it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants