Include burn asset transactions in asset_txs #276
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…output view, and add a distinct on transaction hash
rdlrt
added a commit
that referenced
this pull request
May 4, 2024
## Description <!--- Describe your changes --> Todo: - [x] Update active_stake_cache, no longer requires maintaining account active stake lookup, remove log scan references - [x] Perf improvement for address_txs for 90% of addresses (postgres filtering nuance) - [x] Update cip67_strip_label - [x] Reduce asset info cache rollback to 250 - [x] Update specs (view updates already covered in previous commits, didnt see much to do beyond bumping version) - [x] Retire stake_distribution_new_accounts_update (current query is overkill) - [x] Adapt as per 13.2.0.1 schema - [x] Replace consumed_by_tx_in_id - [x] Replace pool_offline_data - [x] Update epoch references (no longer need 2 epoch wait time) for cache - [x] Remove leftover tx_in references - [x] MIRs (both extract from reserves as well as treasury) are no longer in rewards table, but moved to it's own `instant_rewards` table - update endpoints impacted (I think that's primarily SDC only) - [x] Update `reserve_withdrawals` and `treasury_withdrawals` to include `earned` and `spendable` epochs - [x] Add an endpoint for asset/policy mint history - [x] Retire asset_address_list and asset_policy_info (was deprecated 3 versions ago, and usage-wise had only a single hit in past 48 hours on mainnet) - [x] Fix script_utxos ->> asset_list - [x] New endpoint + cron job to get current protocol parameters - [x] Add is_dangling_delegation check - [x] Specs Testing on guildnet - [x] Update changelog Bugs from testing: - [x] Fix script_utxos, asset_list was not being returned as array , closes #272 - [x] Include burn asset transactions in asset_txs , closes #276 - [x] tx_info: Fix spend_redeemers CTE Join condition, closes #275 - [x] Update Changelog/specs post testing results --------- Co-authored-by: KoT_B_KocMoce <[email protected]> Co-authored-by: XRAY | Robot <[email protected]> Co-authored-by: Scitz0 <[email protected]> Co-authored-by: Greg Beresnev <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Which issue it fixes?
#274