Skip to content

Commit 940916a

Browse files
authored
Merge pull request #140 from CreateThrive/bugfix/users-redirect-error
Bugfix: Redirect when creating a user
2 parents bddff65 + 7baa8c4 commit 940916a

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

src/pages/User/index.jsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,9 @@ const User = () => {
4848
}
4949
}, [isEditing, id, userData, user, dispatch]);
5050

51-
const redirect = (error || success) && <Redirect to={paths.USERS} />;
51+
const redirect = ((isEditing && error) || success) && (
52+
<Redirect to={paths.USERS} />
53+
);
5254

5355
const editUserMessage = useFormatMessage('User.editUser');
5456

src/state/actions/users.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,12 @@ export const fetchUsers = (userId = '') => {
5050
return dispatch(USERS_FETCH_DATA_FAIL({ error }));
5151
}
5252

53+
if (!userData) {
54+
const errorMessage = 'User not available';
55+
toastr.error('', errorMessage);
56+
return dispatch(USERS_FETCH_DATA_FAIL({ error: errorMessage }));
57+
}
58+
5359
const user = { ...userData, id: userId };
5460
const users = getState().users.data;
5561
users.push(user);

0 commit comments

Comments
 (0)