Skip to content
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

Open
maddylewis opened this issue Dec 2, 2024 · 20 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Design External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@maddylewis
Copy link
Contributor

maddylewis commented Dec 2, 2024

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).

image image

Solution

Add "Workspace" in front of all those. So:

  • Workspace profile (LHN)
  • Workspace profile (Page title)
  • Workspace name (name field)

Super basic label updates in the first screenshot:
image

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:
image


Slack thread for additional context - https://expensify.slack.com/archives/C07NZ8B1VTQ/p1732116570767239

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021866126703340321221
  • Upwork Job ID: 1866126703340321221
  • Last Price Increase: 2024-12-09
  • Automatic offers:
    • Krishna2323 | Contributor | 105260187
Issue OwnerCurrent Issue Owner: @JmillsExpensify
@maddylewis maddylewis added Daily KSv2 Design Bug Something is broken. Auto assigns a BugZero manager. labels Dec 2, 2024
Copy link

melvin-bot bot commented Dec 2, 2024

Current assignee @dannymcclain is eligible for the Design assigner, not assigning anyone new.

Copy link

melvin-bot bot commented Dec 2, 2024

Triggered auto assignment to @JmillsExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@maddylewis maddylewis moved this to Polish (LOW) in [#whatsnext] #retain Dec 2, 2024
@Krishna2323
Copy link
Contributor

Krishna2323 commented Dec 2, 2024

Edited by proposal-police: This proposal was edited at 2024-12-02 19:46:02 UTC.

Proposal

Please 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)

@dannymcclain
Copy link
Contributor

Since the building illustration in the page header is a new one, here's the exported SVG:

simple-illustration__building.svg.zip

@Krishna2323
Copy link
Contributor

PROPOSAL UPDATED

  • Pointed the files for illustration change.

@Krishna2323
Copy link
Contributor

PROPOSAL UPDATED

  • Updated main proposal to use existing workspace.common.workspaceName instead of workspace.editor.nameInputLabel.

@melvin-bot melvin-bot bot added the Overdue label Dec 5, 2024
@dannymcclain
Copy link
Contributor

Not overdue Melv. We're just getting started here.

@melvin-bot melvin-bot bot removed the Overdue label Dec 5, 2024
@maddylewis maddylewis added Weekly KSv2 and removed Daily KSv2 labels Dec 5, 2024
@maddylewis
Copy link
Contributor Author

updated the K2 to Weekly as this is more of a polish item.

@Krishna2323
Copy link
Contributor

@dannymcclain, are we ready to make this one external?

@dannymcclain
Copy link
Contributor

Yeah I don't see why not!

@dannymcclain dannymcclain added the External Added to denote the issue can be worked on by a contributor label Dec 9, 2024
@melvin-bot melvin-bot bot changed the title Add "Workspace" to Profile and Name at the workspace level for clarity [$250] Add "Workspace" to Profile and Name at the workspace level for clarity Dec 9, 2024
Copy link

melvin-bot bot commented Dec 9, 2024

Job added to Upwork: https://www.upwork.com/jobs/~021866126703340321221

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 9, 2024
Copy link

melvin-bot bot commented Dec 9, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @eh2077 (External)

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Dec 9, 2024
@dannymcclain
Copy link
Contributor

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?

@dannymcclain dannymcclain added Weekly KSv2 and removed Daily KSv2 labels Dec 9, 2024
@eh2077
Copy link
Contributor

eh2077 commented Dec 9, 2024

@Krishna2323 's proposal looks good to me!

🎀👀🎀 C+ reviewed

Copy link

melvin-bot bot commented Dec 9, 2024

Triggered auto assignment to @carlosmiceli, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 9, 2024
Copy link

melvin-bot bot commented Dec 9, 2024

📣 @Krishna2323 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@Krishna2323
Copy link
Contributor

Krishna2323 commented Dec 10, 2024

@eh2077 @dannymcclain, PR is up for review^

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Dec 26, 2024
@melvin-bot melvin-bot bot changed the title [$250] Add "Workspace" to Profile and Name at the workspace level for clarity [HOLD for payment 2025-01-02] [$250] Add "Workspace" to Profile and Name at the workspace level for clarity Dec 26, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Dec 26, 2024
Copy link

melvin-bot bot commented Dec 26, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented Dec 26, 2024

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:

Copy link

melvin-bot bot commented Dec 26, 2024

@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]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Design External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
Status: Polish (LOW)
Development

No branches or pull requests

6 participants