Youssef Lourayad

Results 56 comments of Youssef Lourayad

This seems to be an issue with PayPal not supporting all currencies that are passed via the URL, for example, this works fine and it displays EUR: https://www.paypal.com/paypalme/kavim/30EUR, or https://www.paypal.com/paypalme/kavim/30GBP,...

Proposal looks to be working. However, I was curious about what we do differently when setting the report's `lastMessageText` optimistically that already unescapes html - since simply sending an `&`...

Sorry I was focused on some high priority tasks that have a tight deadline. I will get to this today. Thanks for the bump @s77rt.

>So I think the new plan is: >1. Fix the bug on `parser.htmlToText` >2. In [getLastVisibleMessageText](https://github.com/Expensify/App/blob/1bc672f8f9e7f057a03d041fb1a3b5cdde789c5e/src/libs/ReportActionsUtils.js#L158) use `text` instead of `html` (and remove the [decoding part](https://github.com/Expensify/App/blob/1bc672f8f9e7f057a03d041fb1a3b5cdde789c5e/src/libs/ReportActionsUtils.js#L159-L160)). This sounds good to...

While fixing in the frontend does work, I think it will be fixing a symptom as this should be fixed in the backend. We should be sending an updated `owner`...

Sounds good @tienifr, let's go with using the `ownerAccountID` as well as updating the policy owner optimistically.

I'm unable to reproduce this one so far, still on it.

This should be fixed once this [PR](https://github.com/Expensify/Auth/pull/9713) deploys.