Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Feb 9, 2021. It is now read-only.

Starting v2 Features #17

Closed
bijanbwb wants to merge 23 commits into master from spike-gameplay-scoring
Closed

Starting v2 Features #17

bijanbwb wants to merge 23 commits into master from spike-gameplay-scoring

Conversation

@bijanbwb
Copy link
Contributor

@bijanbwb bijanbwb commented Jan 13, 2018

  • Fetch, decode, and list gameplays below Elm game.
  • List player's gameplays on player show page.
  • Link from main player scoreboard to player show pages.
  • Adjust Ecto relationships (from many_to_many to has_many)
  • Sync platform content with latest book content.

bijanbwb added 21 commits January 2, 2018 22:12
The demo application and the book had diverged slightly. This commit
should contain all the updates to the Platformer.elm file so that the
demo follows the book content. And this will allow me to push some
additional updates soon.
Not sure if it's a good idea to alias the Accounts module here since I
think the contexts are separated for a reason. But the idea is to sum
the player's total score whenever a new gameplay record is created.
This works for rendering all the gameplays, but the
http://0.0.0.0:4000/api/gameplays endpoint still needs to be fixed so
that it includes the game_id and player_id.
Got stuck pretty bad here for a while. The good news is it seems like a
good start to be able to query for things like `player.games` and
`player.gameplays` and `game.players` and `game.gameplays`.
The schemas seem okay, but I was having a tough time figuring out how
`cast_assoc`, `build_assoc`, and `put_assoc` work. In the current state,
it looks like players and games are properly linked, but the gameplays
get created without loading the associations properly.
@bijanbwb bijanbwb changed the title (削除) Gameplay Scoring (削除ここまで) (追記) Starting v2 Features (追記ここまで) Jan 13, 2018
Copy link
Contributor Author

This branch is now stale, but the features originally spiked out here were merged on the following PRs:

@bijanbwb bijanbwb deleted the spike-gameplay-scoring branch January 15, 2018 04:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /