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

Data model for freezing UTXOs #376

Merged
merged 67 commits into from
Jul 1, 2024
Merged

Data model for freezing UTXOs #376

merged 67 commits into from
Jul 1, 2024

Conversation

markettes
Copy link
Contributor

@markettes markettes commented Jun 25, 2024

  • Adds isFrozen and address to be used in the FMUTXOs
  • Adds tags to FMUTXOs
  • Adds a job to register all the new utxos in the system
- **feat: add address, isFrozen and tag**
- **feat: filter frozen utxos**
- **feat: utxo migration**
- **feat: add postgres for nbxplorer**
- **fix: add migration for optional address and tag**
- **feat: RegisterUTXOs job**
- **feat: add wallet to FMUTXO**
- **feat(FMUTXO.cs): add IsSpent property to determine if UTXO is spent based on channel and withdrawal statuses**
- **feat(FMUTXO.cs): add NBitcoin and NBXplorer dependencies for UTXO model enhancements**
- **feat: consolidate migrations**
- **fix: migration**
- **refactor(ChannelOperationRequestRepositoryTests): add additional null parameter to constructor calls to align with updated constructor signature**
- **Bump deps**
- **Fix problem with new change in nbxplorer when tracking btc addresses**
- **feat: rebuild of the migration**
- **EF Core already does execute the migration in a transaction scope, this was giving a race condition when generating the db (#375)**
- **feat: changes for deps update**
- **Fix AddRangeAsync not using a generic function type making EF Core fail to recognize the entities when saving a list**
- ** Fix  RegisterUTXOsJob to use WalletId instead of Wallet property**
- **fix: rollback changes**
- **feat(FMUTXO.cs): add Blazorise.Extensions for enhanced functionality refactor(FMUTXO.cs): improve IsSpent property logic to handle null collections feat(FUTXORepository.cs): include Tags in FMUTXOs query to provide more detailed data**
- **fix: remove error and set the creation time and the update time**
- **refactor(FUTXORepository.cs): optimize UTXO retrieval logic to enhance performance and accuracy feat(RegisterUTXOsJob.cs): log information after adding new UTXOs to enhance traceability fix(Constants.cs, launchSettings.json): align REGISTER_UTXOS_CRON schedule across configurations for consistency**

markettes and others added 26 commits June 20, 2024 18:37
…el enhancements

feat(FMUTXO.cs): implement Equals method for UTXO comparison with NBitcoin types

refactor(RegisterUTXOsJob.cs): adjust equality check to use modified Equals method in FMUTXO model for clarity and consistency
… parameter to constructor calls to align with updated constructor signature
…is was giving a race condition when generating the db (#375)
…il to recognize the entities when saving a list
refactor(FMUTXO.cs): improve IsSpent property logic to handle null collections
feat(FUTXORepository.cs): include Tags in FMUTXOs query to provide more detailed data
…e performance and accuracy

feat(RegisterUTXOsJob.cs): log information after adding new UTXOs to enhance traceability
fix(Constants.cs, launchSettings.json): align REGISTER_UTXOS_CRON schedule across configurations for consistency
@markettes markettes requested review from Jossec101 and RodriFS and removed request for Jossec101 June 25, 2024 15:41
…clean up code

refactor(FMUTXO.cs): remove redundant Equals method that compares UTXO objects, simplifying the model
…nt method

WHY: Enhance querying capabilities by allowing retrieval of UTXOTag based on both key and outpoint, facilitating more specific data access patterns.
…a integrity

refactor(IUTXOTagRepository): rename GetTagByKeyAndOutpoint to GetByKeyAndOutpoint for consistency

feat(IUTXOTagRepository): add GetByKeyValue method to query UTXOTags by key and value
…structors for consistency

refactor(BitcoinServiceTests.cs): add missing null parameter in CoinSelectionService constructor for completeness
RodriFS
RodriFS previously approved these changes Jun 28, 2024
RodriFS
RodriFS previously approved these changes Jun 28, 2024
src/Helpers/Constants.cs Outdated Show resolved Hide resolved
Jossec101
Jossec101 previously approved these changes Jul 1, 2024
@RodriFS RodriFS dismissed stale reviews from Jossec101 and themself via 750f09a July 1, 2024 11:14
@RodriFS RodriFS merged commit d3d3b1f into main Jul 1, 2024
4 checks passed
@RodriFS RodriFS deleted the utxo-freezing-data-model branch July 1, 2024 11:27
RodriFS added a commit that referenced this pull request Jul 1, 2024
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