Skip to content

[Workspace Feeds] Expensify Card - Card name does not change each time a different member is selected #48376

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

Closed
6 tasks done
IuliiaHerets opened this issue Aug 31, 2024 · 9 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Reviewing Has a PR in review Weekly KSv2

Comments

@IuliiaHerets
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: v9.0.27
Reproducible in staging?: Y
Reproducible in production?: N
Issue was found when executing this PR: Exp #48317
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team

Action Performed:

Precondition:

  • Workspace has bank account set up.
  • Workspace has a few members.
  1. Go to staging.new.expensify.com
  2. Go to Workspace settings > Expensify Card.
  3. Click Issue new card.
  4. Select Member A.
  5. Proceed all the way to the confirmation page (final step) but do not click Issue card button.
    6.Click RHP back button to return to the first step which is member selection page.
  6. Select Member B.
  7. Proceed to "Give it a name" step.
  8. Note that the name is still Member A instead of Member B.
  9. Go to Members.
  10. Click on Member C.
  11. Click New card.
  12. Proceed to "Give it a name" step.
  13. Note that the name is still Member A instead of Member C.

Expected Result:

In Step 9 and 14, the card name should change each time a different user is selected.

Actual Result:

In Step 9 and 14, the card name does not change each time a different user is selected.

Workaround:

Unknown

Platforms:

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6588259_1725054667879.20240831_054036.mp4

View all open jobs on GitHub

@IuliiaHerets IuliiaHerets added DeployBlockerCash This issue or pull request should block deployment Bug Something is broken. Auto assigns a BugZero manager. labels Aug 31, 2024
Copy link

melvin-bot bot commented Aug 31, 2024

Triggered auto assignment to @greg-schroeder (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

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Hourly KSv2 labels Aug 31, 2024
Copy link

melvin-bot bot commented Aug 31, 2024

Triggered auto assignment to @pecanoro (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@daledah
Copy link
Contributor

daledah commented Aug 31, 2024

Proposal

Please re-state the problem that we are trying to solve in this issue.

In Step 9 and 14, the card name does not change each time a different user is selected.

What is the root cause of that problem?

Currently, in the CardNameStep component, we are displaying the cardTitle from Onyx ISSUE_NEW_EXPENSIFY_CARD if it exists.

defaultValue={issueNewCard?.data?.cardTitle ?? defaultCardTitle}

What changes do you think we should make in order to solve the problem?

We should update the cardTitle when the user returns to the step of selecting the assignee and not isEditing here.

        data: {
            assigneeEmail: assignee?.login ?? '',
            cardTitle: isEditing ? issueNewCard.data.cardTitle : '',
        }

And update this condition to show defaultCardTitle when cardTitle is empty string

         defaultValue={issueNewCard?.data?.cardTitle || defaultCardTitle}

What alternative solutions did you explore? (Optional)

NA

@koko57
Copy link
Contributor

koko57 commented Sep 2, 2024

As I am the author of the PR that caused the regression I would like to fix it. It should not be a deploy blocker as the functionality is behind the beta.

cc @mountiny

@mountiny mountiny removed the DeployBlockerCash This issue or pull request should block deployment label Sep 2, 2024
@mountiny
Copy link
Contributor

mountiny commented Sep 2, 2024

Not a blocker this is behind beta

@mountiny
Copy link
Contributor

mountiny commented Sep 2, 2024

@koko57 are you able to fix this in one of the PRs for a clean up

@mountiny mountiny moved this to Release 2.5: SuiteWorld (Sept 9th) in [#whatsnext] #wave-collect Sep 2, 2024
@mountiny mountiny changed the title Expensify Card - Card name does not change each time a different member is selected [Workspace Feeds] Expensify Card - Card name does not change each time a different member is selected Sep 2, 2024
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Sep 2, 2024
@koko57
Copy link
Contributor

koko57 commented Sep 2, 2024

PR opened #48408

@trjExpensify
Copy link
Contributor

This is on prod, closing.

@github-project-automation github-project-automation bot moved this from Release 2.5: SuiteWorld (Sept 9th) to Done in [#whatsnext] #wave-collect Sep 6, 2024
@github-project-automation github-project-automation bot moved this from CRITICAL to Done in [wave-collect] Workspace Feeds Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Reviewing Has a PR in review Weekly KSv2
Projects
Status: Done
Development

No branches or pull requests

9 participants