-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[Due for payment 2025-05-20] [Due for payment 2025-05-13] [$500] Update Migrate tooltips #57045
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
Comments
@flaviadefaria Eep! 4 days overdue now. Issues have feelings too... |
@flaviadefaria Still overdue 6 days?! Let's take care of this! |
This is almost ready to apply all the labels. |
Job added to Upwork: https://www.upwork.com/jobs/~021896560139607289654 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eh2077 ( |
Current assignee @flaviadefaria is eligible for the NewFeature assigner, not assigning anyone new. |
@eh2077 I will unassign you and assign @ishpaul777 as the C+ since he has good context around tooltips and will also be leading the implementation for Convert admin tooltips. |
Last PR is ready for review: |
@flaviadefaria The remaining PRs, which are ready for review, add new tooltips. For now, if the user dismisses them, signs out, and signs in again, these tooltips show again. We need some backend changes so that the new tooltip dismissal persists in the backend. Is it okay to merge these new tooltips without the backend changes, ready so the team can evaluate them, or do we need to wait for an internal engineer to work on the backend changes? |
Let's merge them, but already create an issue to fix it so that an engineer can work on it asap. |
I created the GH and cc'ed you both - #61244 |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.39-8 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 2025-05-13. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
Payment Summary
BugZero Checklist (@flaviadefaria)
|
Upwork job price has been updated to $500 |
Payment Summary: Reviewer: @rayane-d owed $500 via NewDot |
@rayane-d can you please propose regression test steps? Thanks! |
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.44-8 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 2025-05-20. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Payment Summary
BugZero Checklist (@flaviadefaria)
|
I actually think we can skip a regression test for this one. We're likely to make some more adjustments to the tooltips as new product changes go live soon. |
Payment SummaryContributor: @rayane-d owed $500 via NewDot |
Uh oh!
There was an error while loading. Please reload this page.
Problem:
After watching several FS sessions, we've seen that not everyone engaged with the tooltips, and engagement varied depending on whether the user was an admin or an employee.
Solution:
Update our existing set of tooltips so that we have admin and employee specific tooltips that are more relevant to each user role.
Action when clicked: Switches to the Report option on the LHN (screenshot below)
Priority: 2000
Conditions:
role = Admin
(in at least 1 workspace), the user is on the Reports tab and the nudgeMigration.timestamp is not null.[UPDATE] Global Create: 💡 Create expenses, start chatting, and more. Try it out!
Priority 1950 | Conditions:
role = Employee
(in at least 1 workspace), the user can see the global create icon.[UPDATE] Inbox tab: 💡Check what needs your attention and chat about expenses.
Priority 1700
Conditions:
role = All
, the user can see the bottom nav, hasn't been on the Inbox tab yet, and the nudgeMigration.timestamp is not null.[ADD] RBR/GBR chat: 💡You’ll see 🟢 on actions to take, and 🔴 on errors to review.
Priority 1900
Conditions:
role = All
, the user can see a chat with a green or red dot (RBR or GBR)[UPDATE] Workspace chat:💡Chat with approvers
Priority 1800
Conditions:
role = Employee
(in at least 1 workspace), the user is in the Inbox and sees their workspace chat (expense chat)[ADD] Account switcher/copilot access: 💡Access your copilot accounts here
Priority 1600
Conditions:
role = All
, when a user is or becomes a copilot for another account and navigates to Account/Settings page for the first time:[ADD] Outstanding filter: 💡Filter for expenses that need approval
Priority 1925
Conditions:
role = Admin
, InReports
page, with either "Expenses" or "Expense Reports" selected[ADD] Settings tab: 💡Explore your workspace and account settings
Priority 1750
Conditions:
role = Admin
User is in the Inbox or Reports page[ADD] Workspaces tab: 💡View your workspaces
Priority 1550
Conditions:
role = Admin
User is on the Settings page[REMOVE] Concierge chat row and GBR: 💡 Get started here!
[REMOVE] Create in the Workspace chat: 💡 Get started! Submit your expenses here!
[REMOVE] QAB: 💡 Quick action! Just a tap away
[REMOVE] Filter: 💡 Customize your search here!
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @rayane-dThe text was updated successfully, but these errors were encountered: