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] Search - Missing translations for "Search" modal in Spanish #50978

Closed
2 of 8 tasks
IuliiaHerets opened this issue Oct 17, 2024 · 28 comments
Closed
2 of 8 tasks

[$250] Search - Missing translations for "Search" modal in Spanish #50978

IuliiaHerets opened this issue Oct 17, 2024 · 28 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor

Comments

@IuliiaHerets
Copy link

IuliiaHerets commented Oct 17, 2024

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.50-0
Reproducible in staging?: Y
Reproducible in production?: N/A - new feature, doesn't exist in prod
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team

Action Performed:

PreCond: App language is in Spanish

  1. Access staging.new.expensify.com
  2. Sign into a valid account
  3. Tap CMD + K
  4. Scroll down on the Search modal

Expected Result:

User expects everything to be translated to Spanish

Actual Result:

The filters and things like "Track your Expense" are showing in English

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Bug6637228_1729136566559.Search_features_are_not_translated_to_Spanish.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021849836084822280137
  • Upwork Job ID: 1849836084822280137
  • Last Price Increase: 2024-10-25
  • Automatic offers:
    • shubham1206agra | Reviewer | 104641555
Issue OwnerCurrent Issue Owner: @shubham1206agra
@IuliiaHerets IuliiaHerets added DeployBlockerCash This issue or pull request should block deployment Bug Something is broken. Auto assigns a BugZero manager. labels Oct 17, 2024
Copy link

melvin-bot bot commented Oct 17, 2024

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

Copy link

melvin-bot bot commented Oct 17, 2024

💬 A slack conversation has been started in #expensify-open-source

Copy link

melvin-bot bot commented Oct 17, 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.

@melvin-bot melvin-bot bot added the Daily KSv2 label Oct 17, 2024
@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Oct 17, 2024
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.

@IuliiaHerets
Copy link
Author

We think that this bug might be related to #wave-control

@lakchote lakchote added Daily KSv2 and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Oct 17, 2024
@lakchote
Copy link
Contributor

Not a blocker it's missing translations.

@rayane-djouah
Copy link
Contributor

This is not a bug

@rayane-djouah
Copy link
Contributor

We don't translate search queries and results

@lakchote
Copy link
Contributor

cc @luacmartins for your input

@Kicu
Copy link
Contributor

Kicu commented Oct 17, 2024

Exactly what @rayane-djouah said: keywords like type, status I don't see how we could translate - they are part of Search grammar.
Perhaps the values like expense, chat maybe we could translate... but it's still tricky because I'm assuming we couldn't send the translated values to backend.

However displaying different display values to user in the query, but sending untranslated values to backend would be technically possible but will require refactoring how we store the query.
So possibly we could look into this in (or after) 2.5.

@luacmartins
Copy link
Contributor

I agree that the queries are not translated. It seems like we translate the Chat results subtitle/last message if they are a system message, so maybe we can apply the same logic here for the chat results only
Screenshot 2024-10-17 at 9 25 38 AM

@luacmartins luacmartins self-assigned this Oct 17, 2024
@melvin-bot melvin-bot bot added the Overdue label Oct 21, 2024
@greg-schroeder greg-schroeder added the Internal Requires API changes or must be handled by Expensify staff label Oct 21, 2024
@greg-schroeder
Copy link
Contributor

Marking Internal

@melvin-bot melvin-bot bot removed the Overdue label Oct 21, 2024
@ChavdaSachin
Copy link
Contributor

The concierge chat not translated here in video was actually an onboarding task,
and we currently do not translate onboarding tasks.
I have made a proposal here to translate onboarding tasks as well, and in that case this issue also would be solved.

@melvin-bot melvin-bot bot added the Overdue label Oct 24, 2024
@greg-schroeder
Copy link
Contributor

Can you confirm @luacmartins @lakchote ?

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

lakchote commented Oct 25, 2024

Can you confirm @luacmartins @lakchote ?

The problem doesn't pertain to the translation of onboarding tasks, it's related to Search's chat results as a whole.

We need to translate them, if they are a system message.

Regarding the logic, @luacmartins I've found it:
Example #1
Example #2 (implementation)

@luacmartins
Copy link
Contributor

Agreed, I had left a comment about it here

@luacmartins
Copy link
Contributor

luacmartins commented Oct 25, 2024

I think this might be external though.

@luacmartins luacmartins added External Added to denote the issue can be worked on by a contributor and removed Internal Requires API changes or must be handled by Expensify staff labels Oct 25, 2024
@melvin-bot melvin-bot bot changed the title Search - Missing translations for "Search" modal in Spanish [$250] Search - Missing translations for "Search" modal in Spanish Oct 25, 2024
Copy link

melvin-bot bot commented Oct 25, 2024

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

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

melvin-bot bot commented Oct 25, 2024

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

Copy link

melvin-bot bot commented Oct 28, 2024

@lakchote, @luacmartins, @greg-schroeder, @shubham1206agra 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
@shubham1206agra
Copy link
Contributor

@Kicu Would you like to work on this?

@melvin-bot melvin-bot bot removed the Overdue label Oct 29, 2024
@Kicu
Copy link
Contributor

Kicu commented Oct 29, 2024

@shubham1206agra right now I'm focusing on autocomplete features, mainly: #50976 and #50944 + #50943

So at least for the next 2-3 days Im busy. Can take a look after this is done

@lakchote
Copy link
Contributor

This is not urgent @Kicu.

I'm assigning you to work on it when you'll be able to. The autocomplete PRs are more important.

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

melvin-bot bot commented Oct 29, 2024

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

Offer link
Upwork job

@Kicu
Copy link
Contributor

Kicu commented Oct 29, 2024

thanks @lakchote in case someone else on our end has free time then we will ask them to grab this instead.

@SzymczakJ
Copy link
Contributor

SzymczakJ commented Oct 30, 2024

I've investigated it and the Chat results subtitles/last messages that are system messages that are translated in Chat tab are also translated on SearchRouter.
So this comment is technically resolved.
Screenshot 2024-10-30 at 14 44 09
Do we need to do anything else @luacmartins ?

@luacmartins
Copy link
Contributor

Thanks for investigating this. No, I think that was the only pending thing on this issue, so we're good to close. Thank you all!

Copy link

melvin-bot bot commented Oct 30, 2024

@lakchote @luacmartins @greg-schroeder Be sure to fill out the Contact List!

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 Engineering External Added to denote the issue can be worked on by a contributor
Projects
None yet
Development

No branches or pull requests

9 participants