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

CompactBlockProcessor - address TODOs from code base #683

Closed
3 tasks done
HonzaR opened this issue Aug 19, 2022 · 1 comment
Closed
3 tasks done

CompactBlockProcessor - address TODOs from code base #683

HonzaR opened this issue Aug 19, 2022 · 1 comment
Labels
enhancement New feature or request tech_debt Technical debt that should be paid-in-full, eventually.

Comments

@HonzaR
Copy link
Collaborator

HonzaR commented Aug 19, 2022

Is your feature request related to a problem? Please describe.

Filling an additional issue to provide a link to the code base TODO occurrence. This should be ideally split into several more detailed issues once we're on solving it.

TODOs overview:

  • updateRanges: rethink this and make it easier to understand what's happening. Can we reduce this so that we only work with actual changing info rather than periodic snapshots? Do we need to calculate these derived values every time?
  • we still need a way to identify those transactions that failed to be enhanced
  • refreshUtxos: cleanup the way that we prevent this from running excessively. For now, try for about 3 blocks per app launch. If the service fails it is probably disabled on ligthtwalletd, so then stop trying until the next app launch.
@HonzaR HonzaR added enhancement New feature or request tech_debt Technical debt that should be paid-in-full, eventually. labels Aug 19, 2022
@HonzaR
Copy link
Collaborator Author

HonzaR commented Aug 1, 2024

I am closing this as all these ad hoc issues in the code base have been resolved or moved to a separate issue.

@HonzaR HonzaR closed this as completed Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tech_debt Technical debt that should be paid-in-full, eventually.
Projects
None yet
Development

No branches or pull requests

1 participant