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

[1227] Fix broken ToS checkbox handling #1234

Merged
merged 1 commit into from
Apr 26, 2015
Merged

[1227] Fix broken ToS checkbox handling #1234

merged 1 commit into from
Apr 26, 2015

Conversation

orenyk
Copy link
Contributor

@orenyk orenyk commented Apr 24, 2015

Resolves #1227 on master

  • add integration specs for ToS checkbox
  • fix banned user view handling
  • remove check_tos method and only update user if checkout succeeds
  • tweak JS confirmation message on manage reservation page

@esoterik
Copy link
Collaborator

looks good!

Resolves #1227
- add integration specs for ToS checkbox
- fix banned user view handling
- remove check_tos method and only update user if checkout succeeds
- tweak JS confirmation message on manage reservation page
@orenyk
Copy link
Contributor Author

orenyk commented Apr 26, 2015

Thanks, merging!

orenyk added a commit that referenced this pull request Apr 26, 2015
[1227] Fix broken ToS checkbox handling
@orenyk orenyk merged commit ae4aedb into master Apr 26, 2015
@orenyk orenyk deleted the 1227_tos_bug branch October 27, 2015 19:51
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.

Missing ToS checkbox fails silently
2 participants