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

[HOLD #54820] Push notification icon is flaky on iOS #56017

Open
1 of 8 tasks
m-natarajan opened this issue Jan 30, 2025 · 11 comments
Open
1 of 8 tasks

[HOLD #54820] Push notification icon is flaky on iOS #56017

m-natarajan opened this issue Jan 30, 2025 · 11 comments
Assignees
Labels
AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2

Comments

@m-natarajan
Copy link

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:
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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: @AndrewGable
Slack conversation (hyperlinked to channel name): expensify_quality

Action Performed:

  1. Send messages in the same thread
  2. Login as receiver in an iOS device
  3. Observe the notifications

Expected Result:

Notifications received follow same styling

Actual Result:

Different styling notifications received for messages sent from same thread

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

@m-natarajan m-natarajan added AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Needs Reproduction Reproducible steps needed retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause labels Jan 30, 2025
Copy link

melvin-bot bot commented Jan 30, 2025

Triggered auto assignment to @JmillsExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

Copy link

melvin-bot bot commented Jan 30, 2025

Triggered auto assignment to @roryabraham (AutoAssignerNewDotQuality)

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Jan 30, 2025
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@melvin-bot melvin-bot bot added the Overdue label Feb 3, 2025
@muttmuure muttmuure moved this to MEDIUM in [#whatsnext] #quality Feb 3, 2025
Copy link

melvin-bot bot commented Feb 4, 2025

@JmillsExpensify, @roryabraham Huh... This is 4 days overdue. Who can take care of this?

Copy link

melvin-bot bot commented Feb 6, 2025

@JmillsExpensify, @roryabraham 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@roryabraham
Copy link
Contributor

roryabraham commented Feb 6, 2025

Not sure how best to action this one. Going to start by creating a bounty to try and track this one down

@melvin-bot melvin-bot bot removed the Overdue label Feb 6, 2025
@roryabraham
Copy link
Contributor

@arosiclair have you encountered this? Any thoughts?

@arosiclair
Copy link
Contributor

Yeah this is a known issue atm. The push notification styling on both iOS and Android depend on the onyxData in the payload and we remove that when it's too large to fit (usually because the message is too long). We're working on payload compression which should fix that in most cases.

@melvin-bot melvin-bot bot added the Overdue label Feb 10, 2025
Copy link

melvin-bot bot commented Feb 10, 2025

@JmillsExpensify, @roryabraham Whoops! This issue is 2 days overdue. Let's get this updated quick!

Copy link

melvin-bot bot commented Feb 12, 2025

@JmillsExpensify, @roryabraham Huh... This is 4 days overdue. Who can take care of this?

@roryabraham roryabraham changed the title Push notification icon is flaky on iOS [HOLD #54820] Push notification icon is flaky on iOS Feb 12, 2025
@roryabraham roryabraham added Weekly KSv2 and removed Daily KSv2 labels Feb 12, 2025
@melvin-bot melvin-bot bot removed the Overdue label Feb 12, 2025
@roryabraham
Copy link
Contributor

Thanks, @arosiclair. Putting this on HOLD for #54820

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2
Projects
Status: MEDIUM
Development

No branches or pull requests

5 participants