Skip to content
This repository was 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
Loading

  • 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

@Serneum Serneum Jul 28, 2017

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

@Serneum Serneum force-pushed the enhancement/add-games branch from b28f459 to ece2eab Compare July 28, 2017 03:54
@Serneum Serneum force-pushed the enhancement/add-games branch from ece2eab to 516439a Compare July 28, 2017 14:15
@Serneum Serneum force-pushed the enhancement/add-games branch from dd60a55 to 3a5a312 Compare July 28, 2017 14:37
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.

2 participants

@Serneum