Skip to content

Add user authentication#60

Open
swathiiie wants to merge 3 commits into
rspavithra:mainfrom
swathiiie:add-user-authentication
Open

Add user authentication#60
swathiiie wants to merge 3 commits into
rspavithra:mainfrom
swathiiie:add-user-authentication

Conversation

@swathiiie
Copy link
Copy Markdown

Implemented basic user authentication system.

✔ Added user registration and login functionality
✔ Added session persistence using localStorage
✔ Implemented per-user note storage (notes are isolated by username)
✔ Added logout functionality

Each user's notes are now securely separated and accessible only after successful login.

Closes #46
Screenshot (705)

@rspavithra
Copy link
Copy Markdown
Owner

I noticed there is a merge conflict in this PR.
Kindly resolve the conflict and push the updated changes

@rspavithra
Copy link
Copy Markdown
Owner

Screenshot 2026-02-28 232459 Screenshot 2026-02-28 232517

After reviewing and running the project, the following issues were observed:

The login page is not appearing as expected when the app loads.
The authentication flow does not seem to function as described.
The logout button is visible, but it is not working properly.
An image import option has been added, which was not part of the assigned issue.
There are multiple errors/warnings in script.js (showing 9+ issues), which need to be resolved.

Kindly update the PR accordingly.

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.

Add user authentication system with secure login and per-user note isolation

2 participants