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

Completed all tasks #10

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

Conversation

amitmishra11
Copy link

@amitmishra11 amitmishra11 commented Jun 26, 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
  • UI Improvement

Repo link- https://github.com/amitmishra11/csoc22_week3_react

Deployed site- https://csoc22-week3-react-three.vercel.app/login

@shubhangi013
Copy link

shubhangi013 commented Jul 13, 2022

Hey! @amitmishra11 Just a suggestion:

  • you should be more elaborate with your commit messages. Be brief but explanatory enough to convey what you've accomplished.
    Also, add in the links to your repo and deployed site in the PR.

@amitmishra11
Copy link
Author

Hey! @amitmishra11 Just a suggestion:

  • you should be more elaborate with your commit messages. Be brief but explanatory enough to convey what you've accomplished.
    Also, add in the links to your repo and deployed site in the PR.
  • Surely will keep that in mind from next time
  • And now I have added the repo link and deployed site link in the PR.

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