-
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
[HOLD for payment 2025-01-02] [$250] Add "Workspace" to Profile and Name at the workspace level for clarity #53399
Comments
Current assignee @dannymcclain is eligible for the Design assigner, not assigning anyone new. |
Triggered auto assignment to @JmillsExpensify ( |
Edited by proposal-police: This proposal was edited at 2024-12-02 19:46:02 UTC. ProposalPlease re-state the problem that we are trying to solve in this issue.Add "Workspace" to Profile and Name at the workspace level for clarity What is the root cause of that problem?Improvement What changes do you think we should make in order to solve the problem?
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?NA What alternative solutions did you explore? (Optional) |
Since the building illustration in the page header is a new one, here's the exported SVG: |
PROPOSAL UPDATED
|
PROPOSAL UPDATED
|
Not overdue Melv. We're just getting started here. |
updated the K2 to |
@dannymcclain, are we ready to make this one external? |
Yeah I don't see why not! |
Job added to Upwork: https://www.upwork.com/jobs/~021866126703340321221 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eh2077 ( |
Not sure if the bounty should be lower for this though. It seems like a pretty small change to me, but I'm not an engineer so maybe others can weigh in on that? |
@Krishna2323 's proposal looks good to me! 🎀👀🎀 C+ reviewed |
Triggered auto assignment to @carlosmiceli, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
📣 @Krishna2323 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
@eh2077 @dannymcclain, PR is up for review^ |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.78-6 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-01-02. 🎊 For reference, here are some details about the assignees on this issue:
|
@eh2077 @JmillsExpensify @eh2077 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] |
Problem
The use of "Profile" at both the personal account level and at the workspace level. i was configuring the workspace settings, and seeing Profile > Name under the workspace almost prompted me to update that field with my own name (vs. the workspace name).
Solution
Add "Workspace" in front of all those. So:
Super basic label updates in the first screenshot:
Second screenshot has the same label updates but also includes an icon and illustration update to tie into the visual language we typically use for workspaces:
Slack thread for additional context - https://expensify.slack.com/archives/C07NZ8B1VTQ/p1732116570767239
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @JmillsExpensifyThe text was updated successfully, but these errors were encountered: