Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

Completed all tasks #11

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Completed all tasks #11

wants to merge 3 commits into from

Conversation

subodhk01
Copy link
Member

@subodhk01 subodhk01 commented Apr 26, 2020 •
edited
Loading

CSOC Task 1 Submission

https://condescending-varahamihira-7c50fd.netlify.app/
I have completed the following tasks

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

@nishantwrp
Copy link
Member

nishantwrp commented Apr 30, 2020 •
edited
Loading

Hi @subodhk01 ,
I have gone through your app. Here are some suggestions

  • When the does not make any changes in the task name. There should be no request to the backend.

Your score has been updated in the leaderboard.

@nishantwrp nishantwrp added the Judged The Pull Requests which are judged label Apr 30, 2020
@nishantwrp nishantwrp mentioned this pull request May 3, 2020
Open
6 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
Judged The Pull Requests which are judged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@subodhk01 @nishantwrp