issue: ENG-5817
Details
Details
Dispatched the action, checked that dataLoaded and currentUserInfo got updated.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
lib/reducers/user-reducer.js | ||
---|---|---|
68–69 | If we enabled userStore sync before CSAT or at the same time this is fine, but if not we probably want to enable this only after that. There is also an issue about handling older clients trying to update user store in general. I've left a comment about this general issue here. For this specific case it would probably be better to make sessionChange.currentUserInfo an optional field and change the keyserver so that it will only send it to older clients (and finally remove this code completely from the client codebase). |
Comment Actions
Address review - following the discussion in ENG-4730 we want to keep updating the currentUserInfo sent by Ashoat's keyserver