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

downloading an invoice crashes mobile app #7235

Closed
1 task done
jalhubb opened this issue Jul 18, 2024 · 3 comments · Fixed by #7312
Closed
1 task done

downloading an invoice crashes mobile app #7235

jalhubb opened this issue Jul 18, 2024 · 3 comments · Fixed by #7312
Assignees
Labels
android issues that concern the android client but not all clients bug broken functionality, usability problems, unexpected errors ios issues that concern the ios client but not all clients state:tested-with-comments Issue has been tested but some issues were found and needs review.
Milestone

Comments

@jalhubb
Copy link

jalhubb commented Jul 18, 2024

Bug in mobile app

Describe the bug
When downloading an invoice, the app shuts down

To Reproduce
Steps to reproduce the behavior:

  1. Go to Settings > Payment > Invoices & payments
  2. Try to download an invoice
  3. App crashes

Test notes

  • On Android, download an invoice and verify that the application does not crash. Check the device's files, and you should find the invoice present there.
@jalhubb jalhubb added bug broken functionality, usability problems, unexpected errors android issues that concern the android client but not all clients ios issues that concern the ios client but not all clients labels Jul 18, 2024
@rezbyte
Copy link
Contributor

rezbyte commented Jul 22, 2024

Hi,
Thanks for the report.

This might be related to #7160 or maybe "Expected a dict object"?

@paw-hub paw-hub self-assigned this Jul 30, 2024
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in this being thrown.

Fixes #7235
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@paw-hub
Copy link
Contributor

paw-hub commented Jul 30, 2024

I am going to assume that it is related to that, as this is in the sprint now and has the same repro steps and results.

@paw-hub paw-hub linked a pull request Jul 30, 2024 that will close this issue
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
charlag pushed a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
paw-hub added a commit that referenced this issue Jul 30, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
paw-hub added a commit that referenced this issue Aug 2, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
rezbyte pushed a commit that referenced this issue Aug 6, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@rezbyte rezbyte closed this as completed Aug 6, 2024
rezbyte pushed a commit that referenced this issue Aug 6, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
rezbyte pushed a commit that referenced this issue Aug 6, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@rezbyte rezbyte added this to the 240.240807.0 milestone Aug 7, 2024
murilopereirame pushed a commit that referenced this issue Aug 8, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
@rezbyte rezbyte self-assigned this Aug 13, 2024
@rezbyte
Copy link
Contributor

rezbyte commented Aug 13, 2024

This still happens on iOS.

@rezbyte rezbyte added the state:tested-with-comments Issue has been tested but some issues were found and needs review. label Aug 13, 2024
rih-tutao pushed a commit that referenced this issue Aug 15, 2024
It seems we did not specify the mime type for .cmap extensions, thus
downloading invoices can result in an uncaught exception.

Fixes #7235
Fixes #7160
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android issues that concern the android client but not all clients bug broken functionality, usability problems, unexpected errors ios issues that concern the ios client but not all clients state:tested-with-comments Issue has been tested but some issues were found and needs review.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants