Skip to content
This repository was archived by the owner on Jul 24, 2020. It is now read-only.

[1267] Fix Reservation#approved? method #1268

Merged
merged 1 commit into from
May 29, 2015
Merged

Conversation

orenyk
Copy link
Contributor

@orenyk orenyk commented May 28, 2015

Resolves #1267

  • Reservation#approved? method correctly handles open requests
  • add specs for Reservation#approved?

Resolves #1267
- Reservation#approved? method correctly handles open requests
- add specs for Reservation#approved?
@esoterik
Copy link
Collaborator

looks good! (also: oops, should've caught this earlier 😜)

@orenyk
Copy link
Contributor Author

orenyk commented May 29, 2015

Yea... that's why we write tests 😄.

orenyk added a commit that referenced this pull request May 29, 2015
@orenyk orenyk merged commit ea431b5 into master May 29, 2015
@orenyk orenyk deleted the 1267_approved_method branch October 27, 2015 19:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reservation#approved? returns true for open requests
2 participants