Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

WIP: Add Games (DO NOT MERGE) #33

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

WIP: Add Games (DO NOT MERGE) #33

wants to merge 5 commits into from

Conversation

Serneum
Copy link
Member

@Serneum Serneum commented Jul 28, 2017 •
edited

  • Add a Games model
  • Add Games to navbar
  • Tweak generated Games UI
  • Add Games API
  • Add tests around adding/removing games to/from a user's collection

Closes #4

@Serneum Serneum self-assigned this Jul 28, 2017
@Serneum Serneum changed the title Add Games WIP: Add Games (DO NOT MERGE) Jul 28, 2017
Serneum
Serneum commented Jul 28, 2017
View reviewed changes
lib/agoneum/games/game.ex Outdated
@optional_fields ~w(year image)a
@all_fields @required_fields ++ @optional_fields

# TODO: Add a unique constraint for name/year combo
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is done. Need to remove this comment

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@Serneum Serneum

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@Serneum