Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: handle API Key error in reducer #415

Merged
merged 3 commits into from
Jan 20, 2021
Merged

Fix: handle API Key error in reducer #415

merged 3 commits into from
Jan 20, 2021

Conversation

seanyellowd
Copy link
Contributor

proper redux state fixes the bug.
https://jira.openware.work/browse/EMI-320

@seanyellowd seanyellowd added the Ready Ready for a review label Jan 19, 2021
src/modules/user/apiKeys/reducer.ts Outdated Show resolved Hide resolved
@josadcha
Copy link
Contributor

so please, add dataLoaded for actions as I asked you. If post actions are successful redux state should be updated. In ApiKeys container there is a check if data has been loaded

@josadcha josadcha merged commit 4385063 into master Jan 20, 2021
@josadcha josadcha deleted the fix/apikey-error branch January 20, 2021 10:32
calj pushed a commit that referenced this pull request Feb 2, 2021
* Fix: handle api key error in reducer

* Fix: add success value into apiKeys redux state

* Fix: remove unnecessary success redux

Co-authored-by: Sean Lee <5255084-seanlee1@users.noreply.gitlab.com>
oyershov pushed a commit that referenced this pull request Feb 2, 2021
* Fix: handle api key error in reducer

* Fix: add success value into apiKeys redux state

* Fix: remove unnecessary success redux

Co-authored-by: Sean Lee <5255084-seanlee1@users.noreply.gitlab.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready Ready for a review
Development

Successfully merging this pull request may close these issues.

3 participants