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

Dev #149

Merged
merged 8 commits into from
Jun 10, 2020
Merged

Dev #149

merged 8 commits into from
Jun 10, 2020

Conversation

Hassan950
Copy link
Owner

No description provided.

Hassan950 and others added 8 commits June 10, 2020 01:10
* Github service added

* Add github connect test

* Add username to github

Co-authored-by: fuboki <fuboki@pop-os.localdomain>
* Update README.md

* Update README.md

* Added get artists by genre feature

* Completed album.controller tests with coverage 100

* Completed tests for artist and tracks controllers with coverage 100%

* Completed track.service tests

* Added tests for album service and small change in the error logging of notifications

* Update README.md

* Completed album service tests

* Completed artist service tests

* Changed the notification remote db url and added more tests for tracks

* removed unnecessary files and small fix in the document of a validation

* Deleted the attachment of requests in case of refusal

* Removed console log left from debugging

Co-authored-by: Hassan Mohamed <42610032+Hassan950@users.noreply.github.com>
* Github service added

* Add github connect test

* Add username to github

* Refresh token added

* remove func-doc

* ignore functional doc

* Add passport controller tests

* Add reject refresh token

Co-authored-by: fuboki <fuboki@pop-os.localdomain>
* Update README.md

* Update README.md

* Added get artists by genre feature

* Completed album.controller tests with coverage 100

* Completed tests for artist and tracks controllers with coverage 100%

* Completed track.service tests

* Added tests for album service and small change in the error logging of notifications

* Update README.md

* Completed album service tests

* Completed artist service tests

* Changed the notification remote db url and added more tests for tracks

* removed unnecessary files and small fix in the document of a validation

* Update README.md

* Deleted the attachment of requests in case of refusal

* Removed console log left from debugging

* Added functional documentation for notification service and fixed small bugs

Co-authored-by: Hassan Mohamed <42610032+Hassan950@users.noreply.github.com>
* Update README.md

* Update README.md

* Added get artists by genre feature

* Completed album.controller tests with coverage 100

* Completed tests for artist and tracks controllers with coverage 100%

* Completed track.service tests

* Added tests for album service and small change in the error logging of notifications

* Update README.md

* Completed album service tests

* Completed artist service tests

* Changed the notification remote db url and added more tests for tracks

* removed unnecessary files and small fix in the document of a validation

* Update README.md

* Deleted the attachment of requests in case of refusal

* Removed console log left from debugging

* Added functional documentation for notification service and fixed small bugs

Co-authored-by: Hassan Mohamed <42610032+Hassan950@users.noreply.github.com>
@Hassan950 Hassan950 merged commit 2873584 into master Jun 10, 2020
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.

3 participants