Expensify
[$500] Weird push notification when invited to a room with a bunch of members
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 1.4.38-2 Reproducible in staging?: y Reproducible in production?: y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by: @jamesdeanexpensify Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1707336379216449
Action Performed:
- create a room
- Invite a bunch of members to the room
- Observe the push notification as one of the invited member
Expected Result:
Should show the invited members list
Actual Result:
mobile push notification doesn't make sense. we should update the bottom line to "invited you and [NUMBER] others"
Workaround:
unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
- [ ] Android: Native
- [ ] Android: mWeb Chrome
- [x] iOS: Native
- [ ] iOS: mWeb Safari
- [ ] MacOS: Chrome / Safari
- [ ] MacOS: Desktop
Screenshots/Videos
Add any screenshot/video evidence
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~018a2b3f9aac2a663b
- Upwork Job ID: 1755356945680076800
- Last Price Increase: 2024-02-07
Job added to Upwork: https://www.upwork.com/jobs/~018a2b3f9aac2a663b
![melvin-bot[bot] avatar](https://avatars.githubusercontent.com/in/179547?v=4 "Published by a pub.dev verified publisher"){kind=small}
Triggered auto assignment to Contributor-plus team member for initial proposal review - @DylanDylann (External)
Triggered auto assignment to @isabelastisser (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
This issue relates to the notification system. So I think It should be handled on the BE side Send this message to get new eye from internal engineer
🎀 👀 🎀 C+ reviewed
Triggered auto assignment to @danieldoglas, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
@isabelastisser Because this is a BE bug. Could you add the label internal here
yep, this is internal. Also, this is probably the same root cause for https://github.com/Expensify/App/issues/36494. Closing that issue in favor of this one.
Current assignee @DylanDylann is eligible for the Internal assigner, not assigning anyone new.
@isabelastisser I'm unassigning myself, this should be probably a MEDIUM on #vip-vsp
@isabelastisser, @DylanDylann Whoops! This issue is 2 days overdue. Let's get this updated quick!
@isabelastisser @DylanDylann this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!
This issue was included in the VIP project, waiting for an internal engineer assignment.
@isabelastisser, @DylanDylann Whoops! This issue is 2 days overdue. Let's get this updated quick!
This issue was included in the VIP project, waiting for an internal engineer assignment.
@isabelastisser @DylanDylann this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!
Waiting for this:
This MIGHT be solved by https://github.com/Expensify/Web-Expensify/pull/41055 WIP PR.
@isabelastisser @DylanDylann this issue is now 4 weeks old and preventing us from maintaining WAQ. This should now be your highest priority. Please post below what your plan is to get a PR in review ASAP. Thanks!