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-28] Update task title and copy to include link to workspace settings #54977

Open
jamesdeanexpensify opened this issue Jan 8, 2025 · 15 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@jamesdeanexpensify
Copy link
Contributor

jamesdeanexpensify commented Jan 8, 2025

Details

Coming from this convo - Whenever a workspace is created automatically as part of onboarding, the Create a workspace task will be checked off automatically. In those cases, we'd like to update that task in two ways:

  1. In the task title, link on workspace to the workspace settings
  2. In the task itself, update the line "Your new workspace is ready! It'll keep all of your spend (and chats) in one place." to "Your new workspace is ready! [Check it out](link to workspace settings)."

Screenshot

2025-01-08_14-29-15
Issue OwnerCurrent Issue Owner: @laurenreidexpensify
@jamesdeanexpensify jamesdeanexpensify added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jan 8, 2025
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 8, 2025
Copy link

melvin-bot bot commented Jan 8, 2025

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

Copy link

melvin-bot bot commented Jan 8, 2025

Triggered auto assignment to @laurenreidexpensify (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.

Copy link
Contributor

github-actions bot commented Jan 8, 2025

userYour proposal will be dismissed because you did not follow the proposal template.

@Shahidullah-Muffakir
Copy link
Contributor

Shahidullah-Muffakir commented Jan 8, 2025

Proposal

What problem are we trying to solve?

We need to update the "Create a workspace" task to include a clickable link to the workspace settings and improve the task description

What is causing this problem?

Currently, when a workspace is automatically created as part of the onboarding process, the "Create a workspace" task is checked off by default. However, the task title and description don't include a link to the workspace settings. We need to update the task so that it includes this link and also improve the description for better clarity.
Relevant Code

What changes should we make to fix the problem?

  1. Update the task title: Modify the title to include a clickable link to the workspace settings.
  2. Update the task description: Change the description to make the statement about the workspace more informative and clickable.

Here's how the updated code should look:

{
    type: 'createWorkspace',
    autoCompleted: true,
    title: ({ workspaceSettingsLink }) => `Create a [workspace](${workspaceSettingsLink})`,
    description: ({ workspaceSettingsLink }) =>
        '*Create a workspace* to track expenses, scan receipts, chat, and more.\n' +
        '\n' +
        'Here’s how to create a workspace:\n' +
        '\n' +
        '1. Click the settings tab.\n' +
        '2. Click *Workspaces* > *New workspace*.\n' +
        '\n' +
        `*Your new workspace is ready! [Check it out](${workspaceSettingsLink}).`
}
  1. Pass workspaceSettingsLink to the title in this part of the code: Relevant Code

Update it like this:

? task.title({
    integrationName,
    workspaceSettingsLink: `${environmentURL}/${ROUTES.WORKSPACE_INITIAL.getRoute(onboardingPolicyID ?? '-1')}`,
})

  1. Add workspaceSettingsLink to the params types of the title here: Relevant Code
Update the params to include:
params: Partial<{
    integrationName: string;
    workspaceSettingsLink: string;
}>

Note: For the title section, we need to wait for the merge of the PR #54165

What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?

What alternative solutions did you explore? (Optional)

@mananjadhav
Copy link
Collaborator

@Shahidullah-Muffakir's proposal is good but as mentioned we'll need to put it on hold for the Title markdown support PR.

🎀 👀 🎀 C+ reviewed.

Copy link

melvin-bot bot commented Jan 10, 2025

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

@Shahidullah-Muffakir
Copy link
Contributor

Considering that the pending PR #54165 for title markdown support might take longer to get merged, we can move forward by updating the task description first. This way, users will still have a way to navigate to the workspace settings through the description link, even if the title link isn't available yet.

@melvin-bot melvin-bot bot added the Overdue label Jan 13, 2025
@mananjadhav
Copy link
Collaborator

Will let @jasperhuangg and @jamesdeanexpensify take the call on this one.

@jasperhuangg
Copy link
Contributor

@Shahidullah-Muffakir Yeah I like the idea of updating the two in separate PRs for this issue. Let's get the description out first because this is a pretty critical onboarding flow that we should have right.

@melvin-bot melvin-bot bot removed the Overdue label Jan 13, 2025
@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 13, 2025
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Jan 14, 2025
@Shahidullah-Muffakir
Copy link
Contributor

@mananjadhav PR is ready for your review.

@mananjadhav
Copy link
Collaborator

@jasperhuangg The first PR is ready for you.

Copy link

melvin-bot bot commented Jan 17, 2025

⚠️ Looks like this issue was linked to a Deploy Blocker here

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.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Jan 21, 2025
@melvin-bot melvin-bot bot added the Awaiting Payment Auto-added when associated PR is deployed to production label Jan 21, 2025
@melvin-bot melvin-bot bot changed the title Update task title and copy to include link to workspace settings [HOLD for payment 2025-01-28] Update task title and copy to include link to workspace settings Jan 21, 2025
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Jan 21, 2025
Copy link

melvin-bot bot commented Jan 21, 2025

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

Copy link

melvin-bot bot commented Jan 21, 2025

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.87-3 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-28. 🎊

For reference, here are some details about the assignees on this issue:

Copy link

melvin-bot bot commented Jan 21, 2025

@mananjadhav @laurenreidexpensify @mananjadhav 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. External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
Status: No status
Development

No branches or pull requests

5 participants