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

task done #18

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

task done #18

wants to merge 2 commits into from

Conversation

dasari810
Copy link

@dasari810 dasari810 commented Apr 28, 2020 •
edited
Loading

CSOC Task 1 Submission

I have completed the following tasks

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

@Captain-Kirk83 Captain-Kirk83 mentioned this pull request Apr 29, 2020
Open
@sanyud24 sanyud24 mentioned this pull request Apr 29, 2020
Open
6 tasks
@nishantwrp
Copy link
Member

nishantwrp commented May 1, 2020 •
edited
Loading

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

  • No error message when user tries to add empty task.
  • Add task doesn't work.
  • Update task doesn't work.
  • Don't Fetch all tasks again in DELETE.

Your score has been updated in the leaderboard.

@nishantwrp nishantwrp added the Judged The Pull Requests which are judged label May 1, 2020
@dasari810
Copy link
Author

add task and update are working for sometime and not woking for some other times .. when i tested it from my side .. can you please suggest me where iam getting wrong

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
@dasari810 @nishantwrp