Skip to content

HW#8_Trupkina #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
SvetlanaTr wants to merge 28 commits into
base: master
Choose a base branch
from
Open

HW#8_Trupkina #37

SvetlanaTr wants to merge 28 commits into from

Conversation

@SvetlanaTr
Copy link
Collaborator

@SvetlanaTr SvetlanaTr commented Apr 9, 2021 •
edited
Loading

TODO:

  • add
  • mark as done
  • unmark as done
  • delete
    Store data in RAM
  • optional improvement - save in local storage
    Optional improvements:
  • cover the code with unit tests
  • add filter by completed and no (Absent)

Lana Trupkina and others added 28 commits March 25, 2021 23:47
Draft testing
23a6407
HW#7_Trupkina
b956804
add: not to be undefined checking
3e3ac47
Merge branch 'master' of https://github.com/DmitryKorolDevPro/Mentors…
72647e4 
…hipProgramRemote
Inital structure
459ec06
Made all neaded function
af1a577
Made all neaded function
66a28b6
@SvetlanaTr
Delete unit.test.js
a359135
@SvetlanaTr
Delete HW7_Trupkina.js
edced2e
@SvetlanaTr
Delete .gitignore
ebe8288
@SvetlanaTr
Delete package.json
744391c
@SvetlanaTr
Delete package-lock.json
6f960bf
Division into MVC
a71d295
Merge branch 'HW#8_Trupkina' of https://github.com/DmitryKorolDevPro/…
4aaac58 
…MentorshipProgramRemote into HW#8_Trupkina
Checking git
223cd8e
try to separation javascript.js
b30875d
Structure methods calls and define roles of modules
218cbeb
Bug fixing
f7aac70
Next step future filter and test
7313204
Rebuilded with pattern "observer"
454be9b
Small fix
3b30037
tests will be ready soon
bc0bf28
tests will be ready soon
9496465
Finished testing model, next controller
cf9ee3c
almost completely test
6a4ff34
Remove contentLisener
b16c20e
small fix
a243d95
Small fix for controller
1043fbe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@SvetlanaTr