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

Handle undefined IOU Report on IOUdetails page #4534

Merged
merged 3 commits into from
Aug 12, 2021

Conversation

parasharrajat
Copy link
Member

Details

Fixed Issues

$ Fixes #4362

Tests | QA Steps

  1. Open Any chat on newDot which has an outstanding Payment.
  2. Click on the Payment badge from LHN.
  3. Now change the IOU id from URL to arbitrary value.
    for example: http://localhost:8080/iou/details/68438375/rajat

Tested On

  • Web
  • Mobile Web

Screenshots

Web

iou.mp4

@parasharrajat parasharrajat requested a review from a team as a code owner August 10, 2021 22:27
@MelvinBot MelvinBot requested review from chiragsalian and removed request for a team August 10, 2021 22:27
@parasharrajat
Copy link
Member Author

Please confirm the Translations. Thanks

Copy link
Contributor

@chiragsalian chiragsalian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM. Left two copy suggestions.
I'm confirming the Spanish translations with the team so waiting to hear back from them

@@ -336,8 +336,9 @@ export default {
createGroup: 'Create Group',
},
notFound: {
chatYouLookingForCannotBeFound: 'The chat you are looking for cannot be found.',
chatYouLookingForCannotBeFound: 'The chat you are looking for can not be found.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this shouldn't change. It should be,

The chat you are looking for cannot be found.

getMeOutOfHere: 'Get me out of here',
iouReportNotFound: 'Payment details you are looking for can not be found.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rephrase this to be,

The payment details you are looking for cannot be found.

@parasharrajat
Copy link
Member Author

@chiragsalian Changes done.

@parasharrajat
Copy link
Member Author

@chiragsalian Friendly ping.

@chiragsalian
Copy link
Contributor

Yup i've posted about the spanish copy in one of our slack channels and we haven't landed on a resolution. Shall update you as soon as i get an update. Let me bump them.

Copy link
Contributor

@chiragsalian chiragsalian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright they replied, posted.

@@ -336,8 +336,9 @@ export default {
createGroup: 'Crear Grupo',
},
notFound: {
chatYouLookingForCannotBeFound: 'No se pudo encontrar el chat que estabas buscando.',
chatYouLookingForCannotBeFound: 'No se pudo encontrar el chat que está buscando.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you change this to,

El chat que estás buscando no se ha podido encontrar.

getMeOutOfHere: 'Sácame de aquí',
iouReportNotFound: 'No se pudo encontrar los detalles de pago que está buscando.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you change this to,

Los detalles del pago que estás buscando no se han podido encontrar.

@@ -338,6 +338,7 @@ export default {
notFound: {
chatYouLookingForCannotBeFound: 'The chat you are looking for cannot be found.',
getMeOutOfHere: 'Get me out of here',
iouReportNotFound: 'Payment details you are looking for cannot be found.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also the change i suggested here earlier doesn't seem to be applied, Basically, i suggest that it begins with,

The

so the complete text looks like,

The payment details you are looking for cannot be found.

@parasharrajat
Copy link
Member Author

@chiragsalian Done.

Copy link
Contributor

@chiragsalian chiragsalian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you

@chiragsalian chiragsalian merged commit 0c6c942 into Expensify:main Aug 12, 2021
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @chiragsalian in version: 1.0.85-10 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@botify
Copy link

botify commented Aug 25, 2021

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-09-01. 🎊

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.0.86-11 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@botify
Copy link

botify commented Aug 30, 2021

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-09-06. 🎊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Hold - for payment on 2021-08-19] Using an arbitrary ID in IOU Details page causes Infinite Loader
4 participants