Skip to content

Project stats data model updates#369

Merged
heyheyjp merged 4 commits intofeatures/statsfrom
chores/project-stats-model
Aug 4, 2016
Merged

Project stats data model updates#369
heyheyjp merged 4 commits intofeatures/statsfrom
chores/project-stats-model

Conversation

@heyheyjp
Copy link
Collaborator

@heyheyjp heyheyjp commented Aug 4, 2016

Replaces #368, to be merged into feature integration branch features/stats instead of directly into master.

Fixes #365.

Objectives:

  • Group cumulative stats in a single object (we'll be tracking more than ECC now)
  • Make explicit the meaning of property values (also more consistent with other models)
  • Orient player stats in a project-centric way instead of a cycle-centric way

Main Change:

Player stats data previously stored as...

{
  "ecc": 311,
  "cycleProjectECC": {
    "204125f3-ece7-42bb-8a86-a619682ff379": {
      "48382d94-dcd7-479a-9229-7db4bb364916": {
        "abc": 3,
        "ecc": 108,
        "rc": 36,
      },
    } ,
  },
}

...is now stored as...

{
  "stats": {
    "ecc": 311,

    "projects": {
      "48382d94-dcd7-479a-9229-7db4bb364916": {
        "cycles": {
          "204125f3-ece7-42bb-8a86-a619682ff379": {
            "abc": 3,
            "ecc": 108,
            "rc": 36,
          },
        },
      },
    },
  },
}

Other Changes:

  • Renamed stats-related functions referencing ECC specifically to more generic stats-related names

@heyheyjp heyheyjp merged commit 1a6e154 into features/stats Aug 4, 2016
@heyheyjp heyheyjp deleted the chores/project-stats-model branch August 4, 2016 21:53
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.

1 participant

Comments