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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release February 2017 / 1 #245

Merged
merged 40 commits into from Feb 16, 2017

Conversation

Projects
None yet
5 participants
@enricostano
Member

enricostano commented Feb 15, 2017

馃挘

sauloperez and others added some commits Sep 17, 2016

Extract GiveTime service
I only tried to make a transfer and it worked. I followed EDD; going through
each of the errors until it worked, because I didn't find any test for this
action in the controller. Unit testing the service should be the next task.
Do not show give-time link in user's detail page
This disables transfering time to yourself
Do not allow transfers to same account
It adds a validation on the Transfer model and shows a flash error
message when that happens, without making the actual movement.
Do not show any string next to clock on offers.
Due to a bug passing the string argument to #link_to, said string didn't
show up in the page. The way is written now, it shows, so we need to
hide it to be consistent.
Merge pull request #244 from coopdevs/feature/new-menu-proposal
Mejora men煤 para usuarios loggeados

sseerrggii and others added some commits Jan 27, 2017

@sauloperez

馃憦 馃憦

@rewritten rewritten temporarily deployed to timeoverflow-demo Feb 15, 2017 Inactive

@enricostano enricostano merged commit 114eb26 into master Feb 16, 2017

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
@sauloperez

This comment has been minimized.

Show comment
Hide comment
@sauloperez

sauloperez Feb 16, 2017

Contributor

Congrats everyone! looks awesome!

Contributor

sauloperez commented Feb 16, 2017

Congrats everyone! looks awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment