London | 26-ITP-Sep | Mandip Sanger | Sprint 3 | To do List - #1463
mandipsanger wants to merge 2 commits into
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
LonMcGregor
left a comment
There was a problem hiding this comment.
Be careful about your git usage here - this is the todo list task, but you called the branch "alarmclock". Can you resolve that?
Also, did you open this PR before you had fully implemented the task? Can you have another look at the README?
|
in order to solve this i have to make another branch from main and make a fresh PR. I don't think there is another way I have asked abdi he said thhis shoudl be ok. If you really want me to I can change this and you can review. other wise there is no way i can just change the branch from here unless you know any? |
|
OK. @mandipsanger if you want to continue on this branch, that's fine. Just keep in mind the need for an appropriate branch name in future. Could you check and make sure the todo list is complete, then I can review it. |
|
Thank you. ok i will check it it think this. So far it looks ok i will double check the requirements. please check the rest of my things which are complete as I am suppose to finish this before 20 and get badge in order to apply for checkpoint. |
Self checklist
Task code
CYF-1097
Changelist
Todo list app