Skip to content

Fix date picker causing page reload in compliance transaction list #953

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.

Sign up for GitHub

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

Jump to bottom
Merged
TaprootFreak merged 2 commits into from
Feb 6, 2026
Merged

Fix date picker causing page reload in compliance transaction list #953

TaprootFreak merged 2 commits into from
Feb 6, 2026

Conversation

@TaprootFreak
Copy link
Collaborator

@TaprootFreak TaprootFreak commented Feb 6, 2026

Summary

  • Date inputs in /compliance/transactions triggered an API call on every keystroke via useEffect dependencies, which set isLoading=true and replaced the entire page with a spinner — making it impossible to enter dates
  • Extracted the API call into a loadData callback, triggered only on initial load and via a new Search button
  • Loading and error states are now shown inline instead of replacing the filter UI

Test plan

  • Navigate to /compliance/transactions
  • Change date filters — verify the page does NOT reload/flash
  • Click "Search" — verify the API is called with the selected dates
  • Click "Reset" — verify all date fields are cleared
  • Verify initial load still works (spinner shown, then data)
  • Verify CSV export still works with filtered data

@TaprootFreak
Fix date picker causing page reload in compliance transaction list
8954889 
The useEffect triggered on every date change, setting isLoading=true
which replaced the entire page with a spinner, making date inputs
unusable. Now the API call is only triggered via a Search button,
and loading state is shown inline.
@TaprootFreak
Add kycFileId column and fix CHF volume formatting to 2 decimals
5dbfded 
Use toFixed(2) instead of Math.round() in both TX list and KYC files
details to match exact DB precision. Add KycFileId column and filter
checkbox to transaction list.
@TaprootFreak TaprootFreak mentioned this pull request Feb 6, 2026
Merged
2 tasks
@TaprootFreak TaprootFreak marked this pull request as ready for review February 6, 2026 05:02
@TaprootFreak TaprootFreak merged commit 554b6d8 into develop Feb 6, 2026
6 checks passed
@TaprootFreak TaprootFreak deleted the fix/compliance-transaction-list-date-picker branch February 6, 2026 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@davidleomay davidleomay Awaiting requested review from davidleomay davidleomay is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@TaprootFreak