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

All Task Done #20

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

Conversation

Captain-Kirk83
Copy link

CSOC Task 1 Submission

I have completed the following tasks

@nishantwrp
Copy link
Member

Hi @Captain-Kirk83,
I have gone through your app. Here are some suggestions.

  • There should be no request made to the backend when the person is not authenticated.
  • If the username or password is empty. There should be no request to backend.
  • No error when user tries to add empty task.
  • The input box in edit section should have the value of the current task.

Your score has been updated in the leaderboard

@nishantwrp nishantwrp added the Judged The Pull Requests which are judged label May 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Judged The Pull Requests which are judged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants