Skip to content
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.

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

Finished Todo App #3

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

ItayGershman
Copy link

No description provided.

ItayGershman and others added 7 commits February 2, 2022 17:58
This reverts commit e12950d.
* finished the app

* finished the TODO app

* changes in the app according to the PR

* added switch

* fixed the pr comments

Co-authored-by: Itay Gershman <[email protected]>
* webpack implementd

* reverse the list on load - last todo will be displayed as first in the list

* Fixed PR issues
* JSS init

* converting css to jss

* added bundle analyzer into the app

* extract jss imports and setup to style-config file

* fix style files
* HTTP init

* HTTP init after previous PR comments

* extract styles.js to config file, colors file and styles file

* remove bundle-analyzer from webpack.config

* fixed all PR issues
* Converting server to TS

* modify gitignore file and test directory init

* remove .env from repo

* Implementing test environment - drivers, mock of MongoDB. Running tests on get todos and create todo

* update-todos.spec.js is finished

* All tests works, mongoDB base class implemented

* Implemented beforeAndAfterMethod in the testKit

* Fixing comment from Tomben PR

* organizing the application

* fixing Ofir comment - added Generics, improved tests and organized the code

* removed all require from files and switched to import statement

* Implementing reconnection to mongodb

* remove unnecessary console.log
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants