Skip to content

Conversation

@straydogstudio
Copy link

What this does:

  • Adds wins / losses to the player model
  • Adds one convenience method for percentage of wins

Discussion items:

  • The user should not be able to update their wins or losses. How should this be implemented and tested in Pundit?
  • Where in the model would this value be set?
  • Documentation not taken care of. What should be done for these kind of fields that are not parameters but are available on read.

@barelyknown barelyknown deployed to blackjack-api-staging-pr-16 September 17, 2015 18:41 Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants