-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$1000] There is a wrong link in the manage your bills #25962
Comments
ProposalPlease re-state the problem that we are trying to solve in this issue.The wording and hyperlink is abstract. What is the root cause of that problem?This is due to having a
What changes do you think we should make in order to solve the problem?We need to refactor this piece of code such that:
2023-09-14.00-49-31.mp4What alternative solutions did you explore? (Optional)Alternatively, we can:
2023-09-14.00-51-01.mp4 |
Triggered auto assignment to @maddylewis ( |
Bug0 Triage Checklist (Main S/O)
|
Job added to Upwork: https://www.upwork.com/jobs/~014aec13bc46c67f6b |
Current assignee @maddylewis is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @allroundexperts ( |
added external label so existing proposal can be reviewed |
📣 @Al-nitak-external! 📣
|
@maddylewis What is the expected behaviour here? When we click on |
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
ProposalPlease re-state the problem that we are trying to solve in this issue.Clicking on bills from within Settings > Workspaces > Worskspace-Name should show the user a relevant email address along the lines of (<user's-email-domain>@expensify.cash) within a Copy-to-Clipboard interface. What is the root cause of that problem?Basic:For all users whose email addresses contain a public domain (like gmail.com; yahoo.com; publicCompanyWebsite.com; etc), the email text is hard-coded to 1) be "example.com@expensify.cash" instead of a domain specific email address AND 2) to lead to the discussion forum, instead of being within a Copy-to-Clipboard component. Detailed Conjecture:I believe this part was hard-coded by a developer "to be dealt with later", because making the correct resolution for addresses like marc@yahoo.com requires some thought/discussion. Should it be yahoo.com@expensify.cash OR yahoo@expensify.cash OR .... What changes do you think we should make in order to solve the problem?We need to make a well-thought-out resolver function/class here that resolves to an appropriate "bill receiving" email address against all possible user email addresses. For example marc@lemon.edu and ahmed@lemon.edu should resolve into something like lemon-edu@expensify.cash OR lemon@expensify.cash; whichever is architecturally most convenient for Expensify. We need to be mindful of clashes, etc while designing this resolver. |
📣 @talhajavedmukhtar! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
ProposalPlease re-state the problem that we are trying to solve in this issue.When user clicks on Settings > Workspace > Workspace (select a workplace) > Bills, it displays an email address "example.com@expensify.cash". It should show a valid link with CopyToClipboard option but in this case it is showing "example.com@expensify.cash" which leads to a guide to "Receive Company Bills in Expensify". What is the root cause of that problem?The root cause is that the "example.com@expensify.cash" is hardcoded when user.isFromPublicDomain is true which means people coming from email domains such as gmail, yahoo etc (and not company email domains), it shows this hardcoded string along with a link (open in new tab) to a guide "Receive Company Bills in Expensify". What changes do you think we should make in order to solve the problem?It seems like this condition is added on purpose for the two types of accounts (Personal and Company account). The "example" email address is kept here as a placeholder that needs to be changed to a "Guide to submitting bills". What alternative solutions did you explore? (Optional)N/A |
📣 @AbubakerShekhani! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
@allroundexperts - lmk if you have availability to review the proposals for this one! if not, I'll reach out to another C+ to get started. ty! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
sorry, working on a few other things rn but will try to follow up on this before EOD |
alright, sorry for the delay here.
This is expected behavior. That's the instruction to have vendor bills forwarded to someone's Expensify account.
When clicking "View all bills" you should be taken to OldDot > the reports page > and filter bills. I don't have any bills in this account which is why 0 reports show: 2023-09-12_18-22-19.mp4that said, i think this is actually working as expected and any changes to this behavior would touch OldDot / should be considered a new feature request. |
@allroundexperts - based on those details, what do you think? i think we can actually close this one but would love your input. |
Ask your vendors to forward their invoices to example.com@expensify.cash and we'll scan them for you to pay. @maddylewis But when clicking the link it goes to external page which is a blog. Is it expected? |
I think it should not redirect to a blog page when clicking on the link. @maddylewis Can you please confirm this specific detail? If that is expected, then I think we're good to close this! |
Screencast.from.13-09-2023.3.01.45.webm |
Will you tell me exactly what you are clicking that takes you to the community page? but, yes, that's our resource on paying bills in expensify, so that's expected. |
Action Performed: |
Screenshare.-.2023-09-13.9_38_53.PM.mp4 |
Okay, so you're clicking the example email address/the text above it and that's what takes you to the help doc. The only thing I would suggest updating is the hyperlink.
Or |
I feel the same thing. The only thing to do is just update the hyperlink |
This comment was marked as outdated.
This comment was marked as outdated.
Triggered auto assignment to @nkuoch, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
Sorry, this is not the case. @allroundexperts I was the first one to suggest that the link is fine and the text needs to be changed. Check my proposal dated: 28th August, 2023 |
Thanks for pointing out @AbubakerShekhani. I just checked again and it seems like I missed your proposal due to the long conversation happening here. You indeed suggested this first. Let's go with @AbubakerShekhani's proposal. 🎀 👀 🎀 C+ reviewed |
Current assignee @nkuoch is eligible for the choreEngineerContributorManagement assigner, not assigning anyone new. |
Thank you for the prompt response. |
Let's wait for the internal engineer to make the final decision. Then you can proceed to the PR. |
@nkuoch @allroundexperts @maddylewis this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
tbh, i don't think we should update this. the community post that is linked to the existing text is going to be outdated in the next few weeks after the Community to HelpDot migration is complete. we'll be mass-updating a ton of in-product links then, and i think we can just hold off on this one until then. |
closing since we'll be mass updating links in the next couple of weeks. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
There should be a valid link in which the recipients will be sent and a "copy to clipboard" button
Actual Result:
Ask your vendors to forward their invoices to example.com@expensify.cash and we'll scan them for you to pay.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.57-3
Reproducible in staging?: Yes
Reproducible in production?: Yes
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
Notes/Photos/Videos: Any additional supporting documentation
Screenshare.-.2023-08-15.11_17_27.AM.mp4
Expensify/Expensify Issue URL:
Issue reported by: @misgana96
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692087374120429
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: