-
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
[$250] Xero-LHN reports not displayed if login using xero connected a/c #45065
Comments
Triggered auto assignment to @muttmuure ( |
We think this issue might be related to the #vip-vsb |
@muttmuure Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Job added to Upwork: https://www.upwork.com/jobs/~01151d0e3495d1890c |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @jayeshmangwani ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.User sees woo-hoo all caught up message & not LHN reports, if user login using xero integration connected account What is the root cause of that problem?When the user with Xero connection logs in again, We don't have handling for this case yet in the app. What changes do you think we should make in order to solve the problem?We need to add handling for case that the user is required to enable 2FA, we should:
What alternative solutions did you explore? (Optional)In step 5, alternatively we can trigger calling |
@tienifr We have a similar issue #43015 here, that will force workspace admins to enable 2FA before they can use the connection. It looks like this issue will be solved after this PR #44059 is merged. Correct me if I am wrong @c3024 @rushatgabhane. |
@jayeshmangwani yep that's right |
we should close this issue |
Thanks for the confirming Rushat, @muttmuure We can close this issue in favor of #43015 |
@jayeshmangwani, @muttmuure Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Not overdue. Issue can be closed, but Matt is OOO until Tuesday. |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@jayeshmangwani @muttmuure 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! |
@jayeshmangwani, @muttmuure Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@muttmuure We can close this issue. Please check this comment whenever you get a moment. |
Ah yes, let's close. Thanks for the bump! |
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: 9.0.5
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
in mweb
Expected Result:
User must be able to see LHN reports, if user login using xero integration connected account
Actual Result:
User sees woo-hoo all caught up message & not LHN reports, if user login using xero integration connected account
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6535977_1720450241082.vop.1.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @jayeshmangwaniThe text was updated successfully, but these errors were encountered: