From 181036ee5ae653d68363af629002865e9583d324 Mon Sep 17 00:00:00 2001 From: Alexandru Munteanu <alexandru.munt@gmail.com> Date: Thu, 26 Apr 2018 12:47:41 +0300 Subject: [PATCH] fix(invite-reviewers): clear error when showing invite modal --- .../src/components/Reviewers/InviteReviewers.js | 10 +++++++++- packages/components-faraday/src/redux/reviewers.js | 11 +++++++++++ 2 files changed, 20 insertions(+), 1 deletion(-) diff --git a/packages/components-faraday/src/components/Reviewers/InviteReviewers.js b/packages/components-faraday/src/components/Reviewers/InviteReviewers.js index 557a86d74..bd6dc66d4 100644 --- a/packages/components-faraday/src/components/Reviewers/InviteReviewers.js +++ b/packages/components-faraday/src/components/Reviewers/InviteReviewers.js @@ -16,6 +16,7 @@ import { selectReviewersError, selectFetchingReviewers, getCollectionReviewers, + clearReviewersError, } from '../../redux/reviewers' const InviteReviewers = ({ showInviteModal }) => ( @@ -110,11 +111,18 @@ const ModalSwitcher = compose( }) export default compose( + connect(null, { clearReviewersError }), withModal2(props => ({ modalComponent: ModalSwitcher, })), withHandlers({ - showInviteModal: ({ showModal, hideModal, project }) => () => { + showInviteModal: ({ + project, + hideModal, + showModal, + clearReviewersError, + }) => () => { + clearReviewersError() showModal({ type: 'invite-reviewers', collectionId: project.id, diff --git a/packages/components-faraday/src/redux/reviewers.js b/packages/components-faraday/src/redux/reviewers.js index 767836218..05ca433ee 100644 --- a/packages/components-faraday/src/redux/reviewers.js +++ b/packages/components-faraday/src/redux/reviewers.js @@ -66,6 +66,11 @@ const initialState = { reviewers: [], } +const CLEAR_ERROR = 'reviewers/CLEAR_ERROR' +export const clearReviewersError = () => ({ + type: CLEAR_ERROR, +}) + // selectors export const selectReviewers = state => get(state, 'reviewers.reviewers') || [] export const selectReviewersError = state => get(state, 'reviewers.error') @@ -246,6 +251,12 @@ export default (state = initialState, action = {}) => { }, error: null, } + case CLEAR_ERROR: { + return { + ...state, + error: null, + } + } default: return state } -- GitLab