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

More chain index configs #222

Closed

kk-hainq wants to merge 1 commit into IntersectMBO:main from MELD-labs:more-chain-index-configs

Contributor

kk-hainq commented Dec 29, 2021 •

edited

This should resolve #73.

Done:

Add cicStoreNonScriptTxs to allow not storing transactions that do not reference a script.

TODO:

Add cicWhitelistedAddresses to allow only storing transactions that interact with one of the pre-defined addresses.
Write tests.
Write more documentation. However, most documentation efforts should still be done separately for Improve Chain Index documentation #129 once this one is merged.

Pre-submit checklist:

Branch
- Tests are provided (if possible)
- Commit sequence broadly makes sense
- Key commits have useful messages
- Relevant tickets are mentioned in commit messages
- Formatting, materialized Nix files, PNG optimization, etc. are updated
PR
- Self-reviewed the diff
- Useful pull request description
- Reviewer requested


          chain index config: add cicStoreNonScriptTxs

d845fe4

kk-hainq marked this pull request as draft

December 29, 2021 07:38

Contributor Author

kk-hainq commented Mar 10, 2022

We have now realized that this is not a good direction in the near future. One would need a very fast chain index first :(.

kk-hainq closed this

kk-hainq deleted the more-chain-index-configs branch

March 10, 2022 03:39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment