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

[Collect approvers] [$500] Expense - Report submission and approval messages only come through after refreshing the page #34065

Closed
6 tasks done
lanitochka17 opened this issue Jan 5, 2024 · 45 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Reviewing Has a PR in review

Comments

@lanitochka17
Copy link

lanitochka17 commented Jan 5, 2024

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.22-0
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: Applause - Internal Team
Slack conversation:

Action Performed:

Precondition:

  • User is an admin of a Collect workspace on Old Dot
  1. [Old Dot] Create a report with expense
  2. [New Dot] Log in with the same account
  3. [New Dot] Open the expense report from workspace chat
  4. [Old Dot] Submit the report
  5. [Old Dot] Approve the report
  6. [Old Dot] Mark the report as reimbursement
  7. [New Dot] Go to main chat and then click on the preview to return to report

Expected Result:

On New Dot, the system message for report submission and approval will appear in the expense report without refreshing the page

Actual Result:

On New Dot, the system message for report submission and approval only appear in the expense report after refreshing the page. While the reimbursement message appears without issue and without having to refresh the page

Workaround:

Unknown

Platforms:

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

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6333487_1704486086954.33730_desktop.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0137e8422a18862fdd
  • Upwork Job ID: 1743372915513012224
  • Last Price Increase: 2024-01-05
  • Automatic offers:
    • getusha | Reviewer | 28095067
@lanitochka17 lanitochka17 added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jan 5, 2024
Copy link

melvin-bot bot commented Jan 5, 2024

Job added to Upwork: https://www.upwork.com/jobs/~0137e8422a18862fdd

@melvin-bot melvin-bot bot changed the title Expense - Report submission and approval messages only come through after refreshing the page [$500] Expense - Report submission and approval messages only come through after refreshing the page Jan 5, 2024
Copy link

melvin-bot bot commented Jan 5, 2024

Triggered auto assignment to @NicMendonca (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 5, 2024
Copy link

melvin-bot bot commented Jan 5, 2024

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Copy link

melvin-bot bot commented Jan 5, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha (External)

@melvin-bot melvin-bot bot added the Overdue label Jan 8, 2024
@getusha
Copy link
Contributor

getusha commented Jan 8, 2024

Still waiting for proposals

@melvin-bot melvin-bot bot removed the Overdue label Jan 8, 2024
@waterim
Copy link
Contributor

waterim commented Jan 10, 2024

Hello, Im Artem from Callstack and would like to help with this issue

@melvin-bot melvin-bot bot added the Overdue label Jan 11, 2024
@waterim
Copy link
Contributor

waterim commented Jan 11, 2024

After my investigation I found an issue that pusher is not sending a "reportComment" event, it updates the state, but not sending a message.
image

At the same time for REIMBURSED pushed is updating state and also sends a reportComment event.
image

Looks like a backend issue.

@NicMendonca NicMendonca added the Internal Requires API changes or must be handled by Expensify staff label Jan 11, 2024
Copy link

melvin-bot bot commented Jan 11, 2024

Current assignee @getusha is eligible for the Internal assigner, not assigning anyone new.

Copy link

melvin-bot bot commented Jan 11, 2024

Triggered auto assignment to @bondydaa (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@melvin-bot melvin-bot bot removed the Overdue label Jan 11, 2024
@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 11, 2024
Copy link

melvin-bot bot commented Jan 11, 2024

📣 @getusha 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

@bondydaa bondydaa removed the Internal Requires API changes or must be handled by Expensify staff label Jan 11, 2024
Copy link

melvin-bot bot commented Jan 15, 2024

@bondydaa, @NicMendonca, @waterim, @getusha Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Copy link

melvin-bot bot commented Mar 11, 2024

This issue has not been updated in over 15 days. @marcochavezf, @greg-schroeder, @getusha eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

@marcochavezf marcochavezf added Weekly KSv2 and removed Monthly KSv2 labels Mar 12, 2024
@melvin-bot melvin-bot bot removed the Weekly KSv2 label Apr 5, 2024
Copy link

melvin-bot bot commented Apr 5, 2024

This issue has not been updated in over 15 days. @marcochavezf, @greg-schroeder, @getusha eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

@melvin-bot melvin-bot bot added the Monthly KSv2 label Apr 5, 2024
@greg-schroeder
Copy link
Contributor

@marcochavezf can you help me understand where this issue is at? Should this be closed as the PR was deployed a while ago...

@marcochavezf
Copy link
Contributor

Hmm seems there's still an issue here. I will move it to weekly (since it's a polish issue) to check it out accordingly

@marcochavezf marcochavezf added Weekly KSv2 and removed Reviewing Has a PR in review Monthly KSv2 labels Apr 7, 2024
@greg-schroeder
Copy link
Contributor

Okay, thanks!

@melvin-bot melvin-bot bot added the Overdue label Apr 23, 2024
@marcochavezf
Copy link
Contributor

Just finishing a doc and I will come back to this one. Meanwhile I will change it to daily

@melvin-bot melvin-bot bot removed the Overdue label Apr 24, 2024
@marcochavezf marcochavezf added Daily KSv2 and removed Weekly KSv2 labels Apr 24, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 29, 2024
@trjExpensify
Copy link
Contributor

Is this still a bug? It hasn't been retested since Feb, and I can't recall the refresh on some recent testing. Let's clarify if it's still a bug, and what conditions exactly we need to replicate.

Copy link

melvin-bot bot commented Apr 30, 2024

@marcochavezf, @greg-schroeder, @getusha Eep! 4 days overdue now. Issues have feelings too...

@greg-schroeder greg-schroeder added the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label May 1, 2024
@greg-schroeder
Copy link
Contributor

Asking for retest

@melvin-bot melvin-bot bot removed the Overdue label May 1, 2024
@marcochavezf
Copy link
Contributor

I was able to reproduce the approval message from OldDot, working on a fix

@marcochavezf
Copy link
Contributor

PR up

@marcochavezf marcochavezf added the Reviewing Has a PR in review label May 2, 2024
@greg-schroeder
Copy link
Contributor

PR was just deployed

@greg-schroeder
Copy link
Contributor

Automation didn't work - seems this can be closed as no external folks worked on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Reviewing Has a PR in review
Projects
No open projects
Archived in project
Development

No branches or pull requests

9 participants