Skip to content
This repository has been archived by the owner on Nov 15, 2019. It is now read-only.

Added new Idea Domain Events #35

Merged
merged 6 commits into from May 23, 2018
Merged

Added new Idea Domain Events #35

merged 6 commits into from May 23, 2018

Conversation

sgomez
Copy link
Member

@sgomez sgomez commented May 23, 2018

Domain events:

  • IdeaTitleChanged
  • IdeaDescriptionChanged

Closes #24

@coveralls
Copy link

coveralls commented May 23, 2018 •
edited

Pull Request Test Coverage Report for Build 95

  • 21 of 21 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%
Totals Coverage Status
Change from base Build 89: 0.0%
Covered Lines: 165
Relevant Lines: 165
💛 - Coveralls

@sgomez
Copy link
Member Author

sgomez commented May 23, 2018

I forgot to check that name and description really changes before to record the event.

@victormonserrat victormonserrat added this to To do in Domain via automation May 23, 2018
@victormonserrat victormonserrat merged commit af4890a into develop May 23, 2018
Domain automation moved this from To do to Done May 23, 2018
@victormonserrat victormonserrat deleted the feature/issue-24 branch May 23, 2018 10:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@victormonserrat victormonserrat victormonserrat approved these changes

Assignees

@sgomez sgomez

Labels
None yet
Projects
Domain
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@sgomez @coveralls @victormonserrat