diff --git a/src/pages/DetailsPage.js b/src/pages/DetailsPage.js index 2f8d42c686a9..e4a1b763cd62 100755 --- a/src/pages/DetailsPage.js +++ b/src/pages/DetailsPage.js @@ -48,13 +48,10 @@ const propTypes = { /** Route params */ route: matchType.isRequired, - /** Login list for the user that is signed in */ - loginList: PropTypes.shape({ - /** Date login was validated, used to show info indicator status */ - validatedDate: PropTypes.string, - - /** Field-specific server side errors keyed by microtime */ - errorFields: PropTypes.objectOf(PropTypes.objectOf(PropTypes.string)), + /** Session info for the currently logged in user. */ + session: PropTypes.shape({ + /** Currently logged in user accountID */ + accountID: PropTypes.number, }), ...withLocalizePropTypes, @@ -63,7 +60,9 @@ const propTypes = { const defaultProps = { // When opening someone else's profile (via deep link) before login, this is empty personalDetails: {}, - loginList: {}, + session: { + accountID: 0, + }, }; /** @@ -123,7 +122,7 @@ function DetailsPage(props) { const phoneNumber = getPhoneNumber(details); const phoneOrEmail = isSMSLogin ? getPhoneNumber(details) : details.login; - const isCurrentUser = _.keys(props.loginList).includes(details.login); + const isCurrentUser = props.session.accountID === details.accountID; return ( @@ -225,8 +224,8 @@ export default compose( personalDetails: { key: ONYXKEYS.PERSONAL_DETAILS_LIST, }, - loginList: { - key: ONYXKEYS.LOGIN_LIST, + session: { + key: ONYXKEYS.SESSION, }, }), )(DetailsPage); diff --git a/src/pages/ProfilePage.js b/src/pages/ProfilePage.js index b51671341e40..5e1157a29e0d 100755 --- a/src/pages/ProfilePage.js +++ b/src/pages/ProfilePage.js @@ -58,23 +58,25 @@ const propTypes = { /** Route params */ route: matchType.isRequired, - /** Login list for the user that is signed in */ - loginList: PropTypes.shape({ - /** Phone/Email associated with user */ - partnerUserID: PropTypes.string, - }), - /** Indicates whether the app is loading initial data */ isLoadingReportData: PropTypes.bool, + /** Session info for the currently logged in user. */ + session: PropTypes.shape({ + /** Currently logged in user accountID */ + accountID: PropTypes.number, + }), + ...withLocalizePropTypes, }; const defaultProps = { // When opening someone else's profile (via deep link) before login, this is empty personalDetails: {}, - loginList: {}, isLoadingReportData: true, + session: { + accountID: 0, + }, }; /** @@ -122,7 +124,7 @@ function ProfilePage(props) { const phoneNumber = getPhoneNumber(details); const phoneOrEmail = isSMSLogin ? getPhoneNumber(details) : login; - const isCurrentUser = _.keys(props.loginList).includes(login); + const isCurrentUser = props.session.accountID === accountID; const hasMinimumDetails = !_.isEmpty(details.avatar); const isLoading = lodashGet(details, 'isLoading', false) || _.isEmpty(details) || props.isLoadingReportData; @@ -290,9 +292,6 @@ export default compose( personalDetails: { key: ONYXKEYS.PERSONAL_DETAILS_LIST, }, - loginList: { - key: ONYXKEYS.LOGIN_LIST, - }, isLoadingReportData: { key: ONYXKEYS.IS_LOADING_REPORT_DATA, },