Skip to content

feat: add idempotency#118

Merged
nootey merged 2 commits intomainfrom
development
Mar 23, 2026
Merged

feat: add idempotency#118
nootey merged 2 commits intomainfrom
development

Conversation

@nootey
Copy link
Owner

@nootey nootey commented Mar 23, 2026

Proposed changes

  • Add idempotency key to transactions and transfers tables
  • Soft lock client buttons on submit
  • Client change is applicable to all insert forms

Types of changes

What types of changes does your code introduce?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you
did and what alternatives you considered, etc...

@nootey nootey self-assigned this Mar 23, 2026
@nootey nootey added the release:minor Bump minor version (new features) label Mar 23, 2026
@github-actions
Copy link

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/nootey/wealth-warden/internal/http/handlers 0.00% (ø)
github.com/nootey/wealth-warden/internal/models 0.00% (ø)
github.com/nootey/wealth-warden/internal/repositories 0.00% (ø)
github.com/nootey/wealth-warden/internal/services 0.00% (ø)
github.com/nootey/wealth-warden/mocks 0.00% (ø)
github.com/nootey/wealth-warden/pkg/utils 0.00% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/nootey/wealth-warden/internal/http/handlers/transactions_handler.go 0.00% (ø) 0 0 0
github.com/nootey/wealth-warden/internal/models/transaction_models.go 0.00% (ø) 0 0 0
github.com/nootey/wealth-warden/internal/repositories/transaction_repository.go 0.00% (ø) 0 0 0
github.com/nootey/wealth-warden/internal/services/transaction_service.go 0.00% (ø) 0 0 0
github.com/nootey/wealth-warden/mocks/mock_TransactionServiceInterface.go 0.00% (ø) 0 0 0
github.com/nootey/wealth-warden/pkg/utils/ledger_utils.go 0.00% (ø) 0 0 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/nootey/wealth-warden/internal/http/handlers/transactions_handler_test.go
  • github.com/nootey/wealth-warden/internal/services/transaction_service_test.go

@nootey nootey merged commit 59dcbee into main Mar 23, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:minor Bump minor version (new features)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant