Pujan Shah

Results 78 comments of Pujan Shah

I think ND api response should include the outputCurrency data also for the respective policy. If it can't be and needs to be handled on the front-end side then we...

## Reviewer Checklist - [x] I have verified the author checklist is complete (all boxes are checked off). - [x] I verified the correct issue is linked in the `###...

@luacmartins Review text doesn't persist and it instantly turns to Paid, is it fine? https://github.com/user-attachments/assets/e6732d1d-1791-4549-ade3-82949d3642fb I think device A should not go offline bcoz when we turn online it fetches...

Sorry, I will review this in a day.

I am not able to reproduce this issue, @lanitochka17 could you plz recheck and confirm

@Krishna2323's alternate solution of using `setNavigationActionToMicrotaskQueue` seems to be a common pattern to let execute the Onyx update first and then navigation will solve the issue here. We can proceed...

@sakluger I can't reproduce the crash. Also, this isn't a regression from the other issue. https://github.com/user-attachments/assets/a5b2307f-e52e-42a8-ba2b-f56d7386bfae

@puneetlath we can close this issue due to a similar [issue](https://github.com/Expensify/App/issues/52471) has been created earlier and @bernhardoj can transfer their proposal there.

RCA of both issues looks similar to me where the issue remains in deriving the `hasNewestReportAction` due to the different values of last report action created and report lastVisibleActionCreated.

@bernhardoj Why the issue appears only for ios and not for android? Bcoz if it is caused by `hasNewestReportAction` then it should have the same effect on other platforms too.