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

CSOC task 1 #30

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

CSOC task 1 #30

wants to merge 8 commits into from

Conversation

Nazi-pikachu
Copy link

@Nazi-pikachu Nazi-pikachu commented Apr 30, 2020

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

@nishantwrp
Copy link
Member

Hi @Nazi-pikachu,
I have gone through your app. Here are some suggestions.

  • When the username or password is empty there should be no request to the backend.
  • When a person tries to add a empty task there is no error.
  • When a task is added,deleted or edited there should be no page refresh.
  • There should be already the current task in the input task.

Your score has been updated in the leaderboard.

@nishantwrp nishantwrp added the Judged The Pull Requests which are judged label May 2, 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