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

Android - Notification - Tapping notification not opening correct chat #2124

Closed
isagoico opened this issue Mar 27, 2021 · 8 comments
Closed
Assignees
Labels
Improvement Item broken or needs improvement. Weekly KSv2

Comments

@isagoico
Copy link

isagoico commented Mar 27, 2021

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Expected Result:

User is taken to correct chat

Actual Result:

LHN opens and user is displayed bold only a few seconds

Action Performed:

  1. Launch the app and login
  2. Force close the app by swiping out of
  3. Initiate a chat from another device to receive notification
  4. Tap on the notification
  5. Get taken to Chats tab without it opening directly into the chat
  6. Notice that the message preview does not update with new chat line

Workaround:

Unknown

Platform:

Where is this issue confirmed?

Android ✔️

Version Number: 1.0.23-1

Notes/Photos/Videos:

image

image

image

image

Video

Expensify/Expensify Issue URL:

@marcaaron marcaaron added the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Mar 31, 2021
@MelvinBot
Copy link

Triggered auto assignment to @laurenreidexpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@MelvinBot MelvinBot removed the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Mar 31, 2021
@laurenreidexpensify laurenreidexpensify added Daily KSv2 Weekly KSv2 and removed Weekly KSv2 labels Apr 14, 2021
@laurenreidexpensify
Copy link
Contributor

sorry i missed this cos of lack of K2 label

@laurenreidexpensify
Copy link
Contributor

@isagoico 👋🏽
Can you confirm what you mean for step 2 -

  • Force close the app so as to receive the notification

How are you force closing the app?

@laurenreidexpensify
Copy link
Contributor

I'm still testing to replicate this

@laurenreidexpensify
Copy link
Contributor

@isagoico bump on the question above :)

@isagoico
Copy link
Author

Hello Lauren! I'm so sorry I missed this, thought I had actually answered yesterday and seems like I didn't hit the "comment" button.

To force close the app I press the recent button in the navigation bar and then on the trash button to close all open apps. Here the video.

WhatsApp.Video.2021-04-15.at.11.02.45.AM.mp4

@laurenreidexpensify
Copy link
Contributor

Excellent, I'm going to review this on Monday when I have more bandwidth. Thanks @isagoico !

@laurenreidexpensify laurenreidexpensify added Improvement Item broken or needs improvement. External Added to denote the issue can be worked on by a contributor Weekly KSv2 and removed Daily KSv2 External Added to denote the issue can be worked on by a contributor labels Apr 19, 2021
@laurenreidexpensify
Copy link
Contributor

Closing in favour of internal issue here https://github.com/Expensify/Expensify/issues/158830#issuecomment-811182216

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Item broken or needs improvement. Weekly KSv2
Projects
None yet
Development

No branches or pull requests

4 participants