-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[$500] Violation error messages are not shown on the expenses for admin account #35031
Comments
Triggered auto assignment to @stephanieelliott ( |
Job added to Upwork: https://www.upwork.com/jobs/~01431218f64ad15b55 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak ( |
Unable to reproduce in the latest main |
This happens to me consistently as the approver. Example reportID 563335175022642 from rocio@expensify.com. Transaction reportID 3206138938661089. |
@eVoloshchak, @stephanieelliott Huh... This is 4 days overdue. Who can take care of this? |
Still waiting for proposals for this |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Posted in #expensify-swm to see if we can get an expert contrib to jump on this, since it's affecting us internally. https://expensify.slack.com/archives/C04878MDF34/p1706741963165049 |
Hey! I’m Jakub Szymczak from Software Mansion, an expert agency, and I’d like to work on this issue! |
Ah @SzymczakJ the Category and Tag features are only present on Collect workspaces, and not on free workspaces (which is what you created). I'll update the repro steps, but also adding the steps to create the Collect workspace here too. If you run into problems, let me know the email you're using and name of the test workspace so I can try and help!
|
Great, I managed to reproduce the bug and I'm working to fix it. For now, we don't get violations info from backend as admin, but I think it's front end related. |
@stephanieelliott after some research I've found that every update of "request money"(for example change of tag) is triggering a Pusher update that gives new data to UI. All data (transaction data, report data, reportActions data, etc.) is sent through pusher except of transactionViolations data which is responsible for showing violation errors, and that's why we have bugs like this one. I don't have much knowledge about how Pusher works on backend, so I might be mistaken but I think that making Pusher send transactionViolation data when it changes on the backend would solve this issue. |
cc @cead22 on the above. |
Yeah we need to send transaction violations to clients from backend commands like UpdateMoneyRequestAmountAndCurrency, which I have an issue for |
@cead22 could you notify me when transaction violations issue is completed? Then I would check if it solves this bug completely |
@eVoloshchak @stephanieelliott @SzymczakJ this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
Bump @cead22, could you share that transaction violations issue here? Thinking we'll want to hold on that to see if the fix resolves this issue |
The issue is here https://github.com/Expensify/Expensify/issues/361948. I think we can close this issue since we have that one, and the problem in the description of this issue isn't accurate, since violations show for employees, not admins |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.30-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: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1706024549828789
Action Performed:
Expected Result:
violation error messages appear
Actual Result:
No error message shown when login as admin
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?
Screenshots/Videos
Add any screenshot/video evidence
Recording.1705.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: