From 3c1bda19b4802fda6aff9b14699f779f986a8270 Mon Sep 17 00:00:00 2001 From: Bernabe Gonzalez Date: Wed, 5 Feb 2020 12:46:13 -0300 Subject: [PATCH] [BA-171] Remove access token query param from Github request --- src/components/Header/Header.js | 11 ++--------- src/components/Header/HeaderConnected.js | 7 +++---- 2 files changed, 5 insertions(+), 13 deletions(-) diff --git a/src/components/Header/Header.js b/src/components/Header/Header.js index 08520bf..8ead470 100644 --- a/src/components/Header/Header.js +++ b/src/components/Header/Header.js @@ -18,15 +18,9 @@ const isActive = path => strict: false }) -const Header = ({ - user, - accessToken, - fetchUser, - environment, - showLinks = true -}) => { +const Header = ({ user, fetchUser, environment, showLinks = true }) => { useEffect(() => { - !user && fetchUser(accessToken) + !user && fetchUser() }, []) const logout = () => { @@ -98,7 +92,6 @@ const Header = ({ } Header.propTypes = { - accessToken: PropTypes.string.isRequired, fetchUser: PropTypes.func.isRequired, user: PropTypes.object, environment: PropTypes.object, diff --git a/src/components/Header/HeaderConnected.js b/src/components/Header/HeaderConnected.js index c99c016..013663a 100644 --- a/src/components/Header/HeaderConnected.js +++ b/src/components/Header/HeaderConnected.js @@ -8,16 +8,15 @@ import Header from './Header' const userSelector = new UserSelector() const mapState = state => { - const { accessToken } = state.session.user const user = userSelector.getAll(state) - return { accessToken, user, environment: state.environment } + return { user, environment: state.environment } } const mapDispatch = dispatch => { return { - fetchUser: token => + fetchUser: () => dispatch( - request(`/user?access_token=${token}`, { + request('/user', { normalizer: normalizeData, url: 'github' })