Skip to content

feat(export): Share a transaction export from the history screen - #215

Merged
nfebe merged 1 commit into
devfrom
feat/transaction-export
Aug 6, 2026
Merged

feat(export): Share a transaction export from the history screen#215
nfebe merged 1 commit into
devfrom
feat/transaction-export

Conversation

@nfebe

@nfebe nfebe commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

The PDF and Excel items in the transaction history were wired to empty handlers; tapping them closed the menu and did nothing. There is no file-generating package in the app, so this fetches the rendered file from the new export endpoints in trakli/webservice#310 and hands it to the share sheet.

Filters follow what is on screen. Export is refused while signed out or while transactions are still waiting to sync, matching how the reports screen already withholds server stats: the server cannot see unsynced records, so the file would quietly leave them out.

Depends on trakli/webservice#310 being deployed.

The PDF and Excel options in the transaction history opened the menu and
did nothing. They now fetch the file from the server and hand it to the
share sheet, carrying the wallet, category and date filters that are
active on screen.

Exporting is refused while signed out, or while transactions are still
waiting to sync, because the server cannot see those records and the file
would quietly leave them out.
@nfebe
nfebe requested a review from austin047 August 5, 2026 16:51

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@nfebe
nfebe merged commit 9b3c3ab into dev Aug 6, 2026
3 checks passed
@nfebe
nfebe deleted the feat/transaction-export branch August 6, 2026 12:35
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.

3 participants