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

done, some polish left #29

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

Conversation

abhinav-0401
Copy link

@abhinav-0401 abhinav-0401 commented Jul 5, 2022

CSOC Task 3 Submission

I have completed the following tasks

  • Login
  • auth_required.js and no_auth_required.js
  • Add Task
  • Get Tasks
  • Edit Task
  • Delete Task

Here is the deployed app

@NerdyLucifer
Copy link

Please deploy the application and attach the deployment link.
Response is not prompted when the user tries to register or login.
After logging out, the user is not redirect to the login page.
On editing the task, the task gets empty...it would be better if, the there is already the current task as value in the input section.

…edit button the old title for a task remains there instead of being completely empty
@abhinav-0401
Copy link
Author

Please deploy the application and attach the deployment link.

Done.

Response is not prompted when the user tries to register or login.

I'm sorry I didn't quite get what you're trying to say :/

After logging out, the user is not redirect to the login page.

Fixed it.

On editing the task, the task gets empty...it would be better if, the there is already the current task as value in the input section.

Done.

I really appreciate the feedback :D
Thank you :)

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.

2 participants