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

Add message user button #7135

Merged
merged 8 commits into from
Jan 12, 2022
Merged

Add message user button #7135

merged 8 commits into from
Jan 12, 2022

Conversation

luacmartins
Copy link
Contributor

@luacmartins luacmartins commented Jan 11, 2022

Details

Adds a message user button when seeing a user's profile.

Fixed Issues

$ #6661

Tests

  1. Create a group chat.
  2. Click the chat header and select one of the users.
  3. Verify that there is a Message <user> button and clicking it navigates you to the chat with that user.

QA Steps

Steps above.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Screen Shot 2022-01-11 at 4 49 26 PM

Mobile Web

mweb

Desktop

desktop

iOS

ios

Android

android

@luacmartins luacmartins self-assigned this Jan 11, 2022
@luacmartins luacmartins marked this pull request as ready for review January 11, 2022 23:48
@luacmartins luacmartins requested a review from a team as a code owner January 11, 2022 23:48
@MelvinBot MelvinBot requested review from nickmurray47 and removed request for a team January 11, 2022 23:48
@luacmartins
Copy link
Contributor Author

@Expensify/design can I get your eyes on this one? Perhaps the spacing between Local time and Message <user> is too big?

@shawnborton
Copy link
Contributor

I think it looks good personally, nice job!

@luacmartins
Copy link
Contributor Author

Thanks @shawnborton! @nickmurray47 all yours for the review!

@nickmurray47
Copy link
Contributor

taking a look right now!

Copy link
Contributor

@nickmurray47 nickmurray47 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nickmurray47 nickmurray47 merged commit 1d8ce77 into main Jan 12, 2022
@nickmurray47 nickmurray47 deleted the cmartins-dmUser branch January 12, 2022 17:11
@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @nickmurray47 in version: 1.1.27-6 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @nickmurray47 in version: 1.1.27-7 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@Beamanator
Copy link
Contributor

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @Jag96 in version: 1.1.29-5 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

Comment on lines +155 to +161
<MenuItem
title={`${props.translate('common.message')}${details.displayName}`}
icon={Expensicons.ChatBubble}
onPress={() => Report.fetchOrCreateChatReport([props.session.email, details.login])}
wrapperStyle={styles.breakAll}
shouldShowRightIcon
/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR caused a regression #18859. We should show the message button only if you are viewing another profile that is not yours. This was partially fixed by #7228 but it did only check against the current session email and didn't cover the secondary logins.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants