Request to merge feat/588-state-on-user into master
3/3 threads resolved
changed milestone to %"sprint: Jolly Jellyfish"
added MR::Awaiting Review Priority::2 - Normal Product::Wallet Squad::Blue Type::Feature scoped labels
added 2 commits
- 8d2051bc - (fix) Update UserStates Manager test with reward factor fields - #589 (closed)
- 42e5591b - (feat) Add state change value to UserState object - #588 (closed)
added 1 commit
- 59e50c27 - (feat) Split state change view to separate views for increasing and decreasing user state - #589 (closed)
added 6 commits
- 01b11f8d - (fix) Rewards\Contributions: Fix typo in word contribution - #587 (closed)
- c036456b - (fix) Spec bootstrap should load configs (prevents errors connecting to Redis and MQ in dev) - #587 (closed)
- d0425dd2 - (chore) Add missing help descriptions to rewards cli controller - #587 (closed)
- 444c6081 - (feat) Add missing type hints in Contributions Manager - #588 (closed)
- f150613d - (feat) Add action types help text and dry run warning to rewards cli - #587 (closed)
- 18084897 - (feat) Apply the user state reward multiplier to contribution scores - #587 (closed)
Toggle commit listadded 3 commits
- 00beca90 - (fix) Fix expected value in Rewards Manager Spec test
- 9c166d03 - (chore) Add missing command help and dry run option to contributions cli - #587 (closed)
- 3b80c207 - (feat) Add decimal score value to contributions table - #587 (closed)
added 2 commits
- c11424f2 - (fix) Record null value for old contributions score by omitting from INSERT query - #587 (closed)
- d7f0d443 - (feat) Add current reward factor to contributions api - #661 (closed)
added 3 commits
- 167c20ef - (fix) Notification view for user state should be plural rewards_* - #589 (closed)
- d8348319 - (fix) Divide by zero error Contributions\DailyCollection - #587 (closed)
- 6a760f03 - (fix) Send a notification when a user is rewarded with tokens for contributions - #636 (closed)
added 1 commit
- fba7d4e7 - (fix) Test regression in UpdateUserState delegate - #588 (closed)
unmarked as a Work In Progress
DeveloperNOTE: Requires schema changes to
minds.contributions:ALTER TABLE minds.contributions ADD score_decimal decimal;DROP MATERIALIZED VIEW minds.contributions_by_timestamp; CREATE MATERIALIZED VIEW minds.contributions_by_timestamp AS SELECT timestamp, user_guid, metric, amount, score, score_decimal FROM minds.contributions WHERE user_guid IS NOT NULL AND timestamp IS NOT NULL AND metric IS NOT NULL AND amount IS NOT NULL PRIMARY KEY (timestamp, user_guid, metric) WITH CLUSTERING ORDER BY (user_guid ASC, metric ASC);
Resolved by Guy Thouret- Resolved by Guy Thouret
- Resolved by Guy Thouret
added 1 commit
- 2d9337ef - (chore) Convert a rogue tab in whitespace to spaces
resolved all threads
added 1 commit
- 9268f852 - (chore) Update failing UserStates ManagerSpec with new reward factor value - #588 (closed)
approved this merge request
added 34 commits
- 9268f852...d684846f - 32 commits from branch
master - cc1b2a2d - (chore) Result of linting branch before merge - #588 (closed)
- 6458b943 - (chore) Merge linted master to branch - #588 (closed)
- 9268f852...d684846f - 32 commits from branch
added 1 commit
- 8df0dac5 - (chore) More lints that got mised from last commit - #588 (closed)
added 1 commit
- 87e2f5b3 - (chore) Fix test regression after linting - #588 (closed)
added 1 commit
- 0e8d8b00 - (fix) Previously fixed typo made it back into branch - #588 (closed)
added 1 commit
added 10 commits
- af3c0bce...14f32dc5 - 9 commits from branch
master - e491df61 - Merge branch 'master' into feat/588-state-on-user
- af3c0bce...14f32dc5 - 9 commits from branch
added 23 commits
- e491df61...91c223dc - 22 commits from branch
master - 348c3a46 - Merge branch 'master' into feat/588-state-on-user
- e491df61...91c223dc - 22 commits from branch
added 8 commits
- 4cdd24ee - (feat) Simplify UserActivityBuckets - transition detection to cold is not...
- 362df68b - (fix) userActivity data is now sorted by date to guarantee correct ordering - #841
- 46251ea2 - (fix) UserStates - Only allow update of previousState back to ES if it is not empty - #841
- 9825278b - (feat) Add debug option and fakeactivity command to analyics cli - #841
- e4159346 - (feat) Add setTimestamp to Event for generating fake activity in the past - #841
- 1bc69e5d - (fix) Record actual update time in UpdateUserState delegates - #841
- 41149a48 - (feat) Simplify the ActiveUser and UserState Iterators; Use more meaningful...
- 01222a57 - (fix) Allow UserState export to exclude fields with null values to prevent...
Toggle commit listadded 1 commit
added 11 commits
- 168ca91b...72c7048d - 9 commits from branch
master - b6aab277 - (feat) Modify analytics cli and active action event so that activity days...
- 0770cf69 - Merge branch 'master' into feat/588-state-on-user
- 168ca91b...72c7048d - 9 commits from branch
added 1 commit
- 142841ae - (fix) action property of analytics event is not always present
- Developer
Transitions are generating the required notifications with the exception of reward_state_decline_today which should not be telling people that they are declining into resurrected.
added 1 commit
- c6d2641f - (feat) Ignore certain state changes when sending user notifications
added 1 commit
- a88597d7 - (fix) Update spec test for UpdateUserState delegate as state it used does not...
Please register or sign in to reply