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

Implemented ApplyForMembershipUseCase #356

Merged
merged 5 commits into from Apr 25, 2016
Merged

Implemented ApplyForMembershipUseCase #356

merged 5 commits into from Apr 25, 2016

Conversation

@JeroenDeDauw
Copy link
Contributor

JeroenDeDauw commented Apr 24, 2016

@JeroenDeDauw

This comment has been minimized.

Copy link
Contributor Author

JeroenDeDauw commented Apr 24, 2016

The presentation still needs to be done (email and presenter). Unless I'm forgetting about something, I think we can move the remaining tasks (exception handling) to a new ticket with lower priority, once the presentation stuff is done.

Edit: By looking at Phabricator I guess we need to send a mail to the fun people as well.

@JeroenDeDauw JeroenDeDauw changed the title Added peristence to ApplyForMembershipUseCase Implemented ApplyForMembershipUseCase Apr 24, 2016
@gbirke gbirke merged commit 9bc7f31 into master Apr 25, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@gbirke gbirke deleted the ApplyForMembershipUseCase branch Apr 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.