Skip to content

Player project cycle stats DM#372

Merged
heyheyjp merged 13 commits intomasterfrom
features/player-stats-dm
Aug 5, 2016
Merged

Player project cycle stats DM#372
heyheyjp merged 13 commits intomasterfrom
features/player-stats-dm

Conversation

@heyheyjp
Copy link
Collaborator

@heyheyjp heyheyjp commented Aug 5, 2016

NOTE: Forked from features/player-stats (#371).

Fixes #367.
Fixes: #355.

Modified from original UX described in #355.


Retrospective results for #happy-tiger:

Feedback from your team:

  • Jeffrey, I like working with you because you shower every day and I'd like working with you even more if you chewed with your mouth closed.
  • Jeffrey, I like working with you because you have an interesting way of deconstructing problems, and I'd like working with you even more if you took more risks.
  • ...

Stats earned from this project:

Learning Support: 70%
Culture Contribution: 84%

Hours contributed:

Team size: 3
Your hours: 40
All team hours: 120

@deadlyicon (Jared Grippe): 5
@jeffreywescott (Jeffrey Wescott): 42
@tannerwelsh (Tanner Welsh): 34

Contribution to the project:

Self-assessed: 40%
Team-assessed: 45%

Estimated contribution to the project: 42%
Expected contribution for # of hours: 33%
Contribution difference: 9%

screen shot 2016-08-05 at 11 05 22 am

cultureContrbution,
} from '../../server/util/stats'

const QUESTION_TYPES = {
Copy link
Contributor

@jeffreywescott jeffreywescott Aug 5, 2016

Choose a reason for hiding this comment

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

Is this too similar to / a duplicate of server/util/survey.js STATS_QUESTION_TYPES (which is exported)?

Copy link
Collaborator Author

@heyheyjp heyheyjp Aug 5, 2016

Choose a reason for hiding this comment

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

I'd hold off reviewing this until the other is merged. Builds on top of
that - shouldn't be any duplication in these constants now.

I see now, nevermind.

Store additional player project stats
@heyheyjp heyheyjp self-assigned this Aug 5, 2016
@heyheyjp heyheyjp force-pushed the features/player-stats-dm branch from 3d9a645 to 622d11e Compare August 5, 2016 15:48
@heyheyjp heyheyjp merged commit ae6380d into master Aug 5, 2016
@heyheyjp heyheyjp deleted the features/player-stats-dm branch August 5, 2016 18:19
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.

Update cycle completion handling to DM feedback, stats to each player Player DM'ed retrospective summary

2 participants

Comments