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

[Refactor][QBO] Handle better translation keys #41248

Closed
6 tasks
aldo-expensify opened this issue Apr 29, 2024 · 15 comments
Closed
6 tasks

[Refactor][QBO] Handle better translation keys #41248

aldo-expensify opened this issue Apr 29, 2024 · 15 comments
Assignees
Labels
Internal Requires API changes or must be handled by Expensify staff Monthly KSv2 Task

Comments

@aldo-expensify
Copy link
Contributor

aldo-expensify commented Apr 29, 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:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:

@luacmartins suggested to use translateIfPhraseKey for stages

Context: https://expensify.slack.com/archives/C03TQ48KC/p1714421484252409?thread_ts=1714421087.411779&cid=C03TQ48KC

Action Performed:

N/A

Expected Result:

N/A

Actual Result:

N/A

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01065d2b3f47c96271
  • Upwork Job ID: 1785040300946468864
  • Last Price Increase: 2024-04-29
@aldo-expensify aldo-expensify added Daily KSv2 Weekly KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Apr 29, 2024
@aldo-expensify aldo-expensify self-assigned this Apr 29, 2024
Copy link

melvin-bot bot commented Apr 29, 2024

Triggered auto assignment to @isabelastisser (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 removed the Weekly KSv2 label Apr 29, 2024
@aldo-expensify aldo-expensify added Weekly KSv2 Internal Requires API changes or must be handled by Expensify staff and removed Daily KSv2 labels Apr 29, 2024
Copy link

melvin-bot bot commented Apr 29, 2024

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

Copy link

melvin-bot bot commented Apr 29, 2024

Triggered auto assignment to Contributor Plus for review of internal employee PR - @jayeshmangwani (Internal)

@aldo-expensify aldo-expensify added NewFeature Something to build that is a new item. and removed Bug Something is broken. Auto assigns a BugZero manager. labels Apr 29, 2024
Copy link

melvin-bot bot commented Apr 29, 2024

Current assignee @isabelastisser is eligible for the NewFeature assigner, not assigning anyone new.

@aldo-expensify aldo-expensify added Task and removed NewFeature Something to build that is a new item. labels Apr 29, 2024
Copy link

melvin-bot bot commented Apr 29, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

Copy link

melvin-bot bot commented Apr 29, 2024

Triggered auto assignment to Design team member for new feature review - @shawnborton (NewFeature)

@aldo-expensify
Copy link
Contributor Author

Sorry, removing everyone, this is not a Bug , it is just a simple code refactor

@aldo-expensify
Copy link
Contributor Author

No updates yet, focusing on helping with the fire

@melvin-bot melvin-bot bot removed the Overdue label May 8, 2024
@melvin-bot melvin-bot bot added the Overdue label May 16, 2024
@aldo-expensify
Copy link
Contributor Author

sames as before

@melvin-bot melvin-bot bot removed the Overdue label May 17, 2024
@melvin-bot melvin-bot bot added the Overdue label May 27, 2024
@aldo-expensify
Copy link
Contributor Author

sames as before

@melvin-bot melvin-bot bot removed the Overdue label May 29, 2024
@melvin-bot melvin-bot bot added the Overdue label Jun 6, 2024
@aldo-expensify
Copy link
Contributor Author

sames as before

@melvin-bot melvin-bot bot removed the Overdue label Jun 7, 2024
@aldo-expensify aldo-expensify added Monthly KSv2 and removed Weekly KSv2 labels Jun 7, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 8, 2024
@aldo-expensify
Copy link
Contributor Author

Coming back from being OOO, I'll try to put some time on this this week.

@melvin-bot melvin-bot bot removed the Overdue label Jul 16, 2024
@melvin-bot melvin-bot bot added the Overdue label Aug 16, 2024
@aldo-expensify
Copy link
Contributor Author

aldo-expensify commented Aug 17, 2024

No progress yet

@melvin-bot melvin-bot bot removed the Overdue label Aug 17, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 17, 2024
@aldo-expensify
Copy link
Contributor Author

No progress yet

@melvin-bot melvin-bot bot removed the Overdue label Sep 19, 2024
@melvin-bot melvin-bot bot added the Overdue label Oct 21, 2024
@aldo-expensify
Copy link
Contributor Author

The function translateIfPhraseKey doesn't exist anymore. I search for something similar in our code and didn't find.

I looked into the en.ts file and there are already a few other cases where we have a function with a switch to handle translation.

I'm ok with the current design and using en.ts because it keeps the translations in the same file and it is pretty readable IMO, so I'm just going to close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal Requires API changes or must be handled by Expensify staff Monthly KSv2 Task
Projects
None yet
Development

No branches or pull requests

4 participants