Abdelhafidh Belalia
Abdelhafidh Belalia
Any new commits should trigger the test but it seems unstable, even after merging main it could still fail
This may not work as you'd expect, in our case we had ``s rendered inside `` which is semantically incorrect.
On Web the pdf preview have a cursor pointer, let's use the default pointer instead as the preview is not clickable https://github.com/Expensify/App/assets/16493223/97465eb9-fc3e-427d-9ba0-42715d19fd2b
The local preview do not match the one returned by the server https://github.com/Expensify/App/assets/16493223/e084b2cd-9e4d-4bb4-a97d-ddb997b98c84 https://github.com/Expensify/App/assets/16493223/92c69be3-1c9a-4bb3-b402-127db5938f62
Uploading a password protected pdf results in a non-ending prompt unless you type the right pasword https://github.com/Expensify/App/assets/16493223/022b1852-7608-4ba6-bce1-9c9c84e25fb5
@eh2077 Please check https://github.com/Expensify/App/pull/35255#discussion_r1471656045 For the password bug, I think we should show an error if the user is uploading a password protected file since the BE can't scan it...
@eh2077 Any updates here?
@eh2077 Regarding the password issue, previously the uploaded file would just fail but now it's creating an endless prompt which is something that we should fix. The least we can...
@eh2077 Any updates here?
@eh2077 Please do not force push. This messes up the review process. Can you try undo that force push?