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

[$250] Chat - Invalid welcome text when opening a conversation with a new user #51247

Closed
3 of 8 tasks
IuliiaHerets opened this issue Oct 22, 2024 · 19 comments
Closed
3 of 8 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors

Comments

@IuliiaHerets
Copy link

IuliiaHerets commented Oct 22, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 9.0.52-1
Reproducible in staging?: Y
Reproducible in production?: Y
Issue was found when executing this PR: #49592
Issue reported by: Applause Internal Team

Action Performed:

  1. Navigate to https://staging.new.expensify.com/
  2. Log in with a Gmail account
  3. Navigate to FAB - Start chat
  4. Input and enter 1:1 DM with a Gmail account that you never had contact with

Expected Result:

Welcome should be "Say hello! This is the beginning of your chat with username. You can also use the + button to slip an expense, submin an expense, pay expenses, or assign a task!"

Actual Result:

Welcome text is "Say hello! This chat is with username. Use the + button to split, submit, or pay an expense."

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6641983_1729597990838.bandicam_2024-10-22_13-47-38-490.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021849547431797589961
  • Upwork Job ID: 1849547431797589961
  • Last Price Increase: 2024-11-07
Issue OwnerCurrent Issue Owner: @ZhenjaHorbach
@IuliiaHerets IuliiaHerets added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Oct 22, 2024
Copy link

melvin-bot bot commented Oct 22, 2024

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

@IuliiaHerets
Copy link
Author

@zanyrenney FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@nyomanjyotisa
Copy link
Contributor

nyomanjyotisa commented Oct 22, 2024

Edited by proposal-police: This proposal was edited at 2024-10-25 00:25:20 UTC.

Proposal

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

Chat - Invalid welcome text when opening a conversation with a new user

What is the root cause of that problem?

Changes Request

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

We need to update the reportActionsView on here and here

    reportActionsView: {
        ....
        beginningOfChatHistory: 'This is the beginning of your chat with ',
        ...
        usePlusButton: ({additionalText}: UsePlusButtonParams) => `\nYou can also use the + button to ${additionalText}!`,
        askConcierge: '\nAsk questions and get 24/7 realtime support.',
        iouTypes: {
            pay: 'pay',
            split: 'split',
            submit: 'submit',
            track: 'track',
            invoice: 'invoice',
        },
        anExpense: 'an expense',
        assignTask: 'assign a task',
    },

and update the additionalText on here

    const additionalText = `${filteredOptions.map((item) => `${translate(`reportActionsView.iouTypes.${item}`)} ${translate('reportActionsView.anExpense')}`).join(', ')} ${translate(
        'common.or',
    )} ${translate('reportActionsView.assignTask')}`;
    },

RESULT

image

What alternative solutions did you explore? (Optional)

Or just update the reportActionsView to

    reportActionsView: {
        ...
        beginningOfChatHistory: 'This is the beginning of your chat with ',
        ...
        usePlusButton: ({additionalText}: UsePlusButtonParams) => `\nYou can also use the + button to ${additionalText}, or assign a task!`,        
        askConcierge: '\nAsk questions and get 24/7 realtime support.',
        iouTypes: {
            pay: 'pay expenses',
            split: 'split an expense',
            submit: 'submit an expense',
            track: 'track an expense',
            invoice: 'invoice an expense',
        },
    },

and update the additionalText

const additionalText = filteredOptions.map((item) => translate(`reportActionsView.iouTypes.${item}`)).join(', ');

@nkdengineer
Copy link
Contributor

The welcome text is expected here

@zanyrenney
Copy link
Contributor

Well done for forgetting the expected welcome text.

@melvin-bot melvin-bot bot removed the Overdue label Oct 24, 2024
@zanyrenney zanyrenney added External Added to denote the issue can be worked on by a contributor Overdue labels Oct 24, 2024
@melvin-bot melvin-bot bot changed the title Chat - Invalid welcome text when opening a conversation with a new user [$250] Chat - Invalid welcome text when opening a conversation with a new user Oct 24, 2024
Copy link

melvin-bot bot commented Oct 24, 2024

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

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

melvin-bot bot commented Oct 24, 2024

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

@melvin-bot melvin-bot bot removed the Overdue label Oct 24, 2024
@ZhenjaHorbach
Copy link
Contributor

ZhenjaHorbach commented Oct 25, 2024

#49592

Hmmm
Yes
Looks like this text is expected

@zanyrenney
Could you please check this issue ?

And we can close this issue or update the text otherwise

Copy link

melvin-bot bot commented Oct 28, 2024

@zanyrenney, @ZhenjaHorbach Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@melvin-bot melvin-bot bot added the Overdue label Oct 28, 2024
@ZhenjaHorbach
Copy link
Contributor

Not overdue

@melvin-bot melvin-bot bot removed the Overdue label Oct 28, 2024
Copy link

melvin-bot bot commented Oct 31, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@melvin-bot melvin-bot bot added the Overdue label Oct 31, 2024
@ZhenjaHorbach
Copy link
Contributor

#49592

Hmmm Yes Looks like this text is expected

@zanyrenney Could you please check this issue ?

And we can close this issue or update the text otherwise

@zanyrenney
Friendly bump !

@melvin-bot melvin-bot bot removed the Overdue label Oct 31, 2024
Copy link

melvin-bot bot commented Nov 5, 2024

@zanyrenney @ZhenjaHorbach this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@melvin-bot melvin-bot bot added the Overdue label Nov 5, 2024
Copy link

melvin-bot bot commented Nov 5, 2024

@zanyrenney, @ZhenjaHorbach Huh... This is 4 days overdue. Who can take care of this?

Copy link

melvin-bot bot commented Nov 7, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

Copy link

melvin-bot bot commented Nov 12, 2024

@zanyrenney, @ZhenjaHorbach 10 days overdue. I'm getting more depressed than Marvin.

@zanyrenney
Copy link
Contributor

Ooh sorry @ZhenjaHorbach

@zanyrenney
Copy link
Contributor

Checked in on the issue here I agree this looks like it is expected behaviour as per James' table of copy updates, I agree we can close this out.

@melvin-bot melvin-bot bot removed the Overdue label Nov 13, 2024
@ZhenjaHorbach
Copy link
Contributor

Ooh sorry @ZhenjaHorbach

No problem 😅
And thanks for checking !

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. Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors
Projects
None yet
Development

No branches or pull requests

5 participants