Open IuliiaHerets opened 1 week ago
Triggered auto assignment to @strepanier03 (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.
@strepanier03 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
Chat - There is no closing question mark on delete comment confirmation modal in Spanish
We are not adding ? here https://github.com/Expensify/App/blob/99f280b0edae75ff59614a1c5e98c47b39800aa5/src/languages/es.ts#L636
We should add it like this
deleteConfirmation: ({action}: DeleteConfirmationParams) =>
`¿Estás seguro de que quieres eliminar este ${action?.actionName === CONST.REPORT.ACTIONS.TYPE.IOU ? 'gasto?' : 'comentario?'}`,
Edited by proposal-police: This proposal was edited at 2024-10-05 15:54:14 UTC.
There is no closing question mark on delete comment confirmation modal in Spanish
this translation is not correct https://github.com/Expensify/App/blob/87ea10031ba24d3fa3a5d3c71cc4de030085d4cf/src/languages/es.ts#L636-L637
add question mark to the end
deleteConfirmation: ({action}: DeleteConfirmationParams) =>
`¿Estás seguro de que quieres eliminar este ${action?.actionName === CONST.REPORT.ACTIONS.TYPE.IOU ? 'gasto' : 'comentario'}?`,
Added the exact code changes
Job added to Upwork: https://www.upwork.com/jobs/~021843383751880246894
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eh2077 (External
)
Updated the job post and the title as this change is very simple and small, and we agreed to lower the price on this one.
We are missing a "?" at the end of delete comment confirmation modal's spanish text.
In the src/languages/es.ts
file we ask for delete confirmation without the "?":
deleteConfirmation: ({action}: DeleteConfirmationParams) =>
`¿Estás seguro de que quieres eliminar este ${action?.actionName === CONST.REPORT.ACTIONS.TYPE.IOU ? 'gasto' : 'comentario'}`,
Add the missing "?" in the end of the string in the translation file.
📣 @parotax! 📣 Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork. Please follow these steps:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>
This might be a dupe of https://github.com/Expensify/App/issues/49893, going to check and if so close this out.
Contributor details Your Expensify account email: luukas.k.lehtinen@gmail.com Upwork Profile Link: https://www.upwork.com/freelancers/~01f29577390fed6ca6
✅ Contributor details stored successfully. Thank you for contributing to Expensify!
This might be a dupe of #49893, going to check and if so close this out.
@strepanier03 To me it doesn't seem to be a duplicate. It is just a similar issue in the "Download attachment" modal. This is in "Delete message" modal.
Thank you for your speedy proposals!
This one is really straightforward. I think we should go with @MuaazArshad 's proposal as they're the first to point the root cause.
@MuaazArshad posted at 2024-10-05T15:51:45Z
while @abzokhattab posted at 2024-10-05T15:54:03Z
🎀👀🎀 C+ reviewed
Triggered auto assignment to @neil-marcellini, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
Thank you for your speedy proposals!
This one is really straightforward. I think we should go with @MuaazArshad 's proposal as they're the first to point the root cause.
I agree, sounds good. @strepanier03 this one is so simple that I would suggest we pay $0 for it, or at least next time we have a similar copy change. No C+ is really needed either.
📣 @MuaazArshad You have been assigned to this job! Please apply to the Upwork job and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻 Once you apply to this job, your Upwork ID will be stored and you will be automatically hired for future jobs! Keep in mind: Code of Conduct | Contributing 📖
@strepanier03, @neil-marcellini, @MuaazArshad, @eh2077 Eep! 4 days overdue now. Issues have feelings too...
The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.48-2 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2024-10-22. :confetti_ball:
For reference, here are some details about the assignees on this issue:
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
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: v9.0.45-2 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5045504 Email or phone of affected tester (no customers): applausetester+vd_web100424@applause.expensifail.com Issue reported by: Applause Internal Team
Action Performed:
Pre-requisite: the user must have set the language preference to Spanish.
Expected Result:
Text on confirmation modal should say: ¿Estás seguro de que quieres eliminar este comentario?
Actual Result:
Text on confirmation modal says: ¿Estás seguro de que quieres eliminar este comentario
Workaround:
Unknown
Platforms:
Screenshots/Videos
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @strepanier03