-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 for payment 2024-12-05] [$50] Update copy on the wallet settings page #52789
Comments
Triggered auto assignment to @trjExpensify ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Update copy on the wallet settings page What is the root cause of that problem?This is an improvment What changes do you think we should make in order to solve the problem?Change this translation to Line 1410 in 763215e
change this to Line 1412 in 763215e
We also need to update the translation in es.ts file accordingly What alternative solutions did you explore? (Optional) |
@tgolen did you want to send this external? If so, I think we reduce the price quite considerably to say $50 for a small copy change. |
I think a reduced price makes a lot of sense! |
|
Job added to Upwork: https://www.upwork.com/jobs/~021859017641475961132 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rayane-djouah ( |
Contributor details |
📣 @hungdannt! 📣
|
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.Users find it difficult to navigate the wallet settings page due to cluttered interface copy. What is the root cause of that problem?The current wallet section title and subtitle are overly verbose and redundant. What changes do you think we should make in order to solve the problem?Update translations in walletPage: {
sendAndReceiveMoney: 'Expensify Wallet', // Changed from "Send and receive money with friends"
walletEnabledToSendAndReceiveMoney: 'Send and receive money with friends', // Changed from "Your wallet has been enabled to send and receive money with friends"
} What alternative solutions did you explore? (Optional) |
ProposalPlease re-state the problem that we are trying to solve in this issue.Clean up some of the copy on the wallet settings page: What is the root cause of that problem?UX improvement suggestion to reduce interface clutter and improve customer engagement and retention What changes do you think we should make in order to solve the problem?
What alternative solutions did you explore? (Optional)The current solution assumes that the title and subtitle for Wallet section remains the same in both activated and non-activated condition. |
@tgolen @trjExpensify - In the case where the wallet is not enabled, I think we need to update the title from ![]() |
Hm, I don't like changing the title personally. I would think we just have this in both pre/post-enablement states? Expensify Wallet |
Agreed, having this in both pre/post-enablement states makes sense for consistency. Thanks! |
@prakashbask's proposal looks good to me. 🎀👀🎀 C+ reviewed |
Current assignee @tgolen is eligible for the choreEngineerContributorManagement assigner, not assigning anyone new. |
📣 @rayane-djouah 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
📣 @prakashbask You have been assigned to this job! |
I have applied to the job on Upwork and target to have the PR ready by 22-Nov |
PR #52942 |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.67-9 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-12-05. 🎊 For reference, here are some details about the assignees on this issue:
|
@rayane-djouah @trjExpensify @rayane-djouah The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
BugZero Checklist:
|
Agreed, small copy update. Payment summary as follows:
|
Paid @prakashbask, closing! |
Problem
When users find it difficult to navigate our application's features due to interface clutter, it leads to decreased engagement and lower conversion and retention rates.
Solution
Clean up some of the copy on the wallet settings page:
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @trjExpensifyThe text was updated successfully, but these errors were encountered: