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

Send twit after a new course created #90

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

Conversation

mgzamora
Copy link

Practice for Arquitectura Hexagonal Mooc

Added a use case which will publish a twit message on Twiter.

* course finder refactor to domain layer

* missing config

* rename course mother function and update references

* removed unused class

* added rename to domain class and create course renamed event

* added test to course rename

* added test to course rename domain model

* added rename course handler and tests

* added controller and route

* added dependency injection config

* added test scenario
* added some semantic to rename course command handler test

* added change description command

* added description mother class and import where  necessary

* added description to course class

* modify tests to admit new course property
* rename getCourseResponse to CourseResponse

* added get courses query

* added get courses controller and response

* added add course to environment arranger

* fix scenario tests

* added course repository get all and tests

* added get courses handler application uc

* added get courses handler application tests

* add course with id replace add course on arranger

* add get courses test scenarios

* added get courses route

* added dependency injection config

* fix test
* added scenario, controller and command

* added userId

* added like functionality to course

* added structure and tests

* fix feature scenario
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.

None yet

1 participant