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

Merge json-uri optional arg #64

Merged
merged 30 commits into from
Jun 22, 2023
Merged

Merge json-uri optional arg #64

merged 30 commits into from
Jun 22, 2023

Conversation

tahsintunan
Copy link

No description provided.

NicolasPennie and others added 29 commits April 5, 2023 19:06
Fix GetAssetsByCreator query
Handle and log http errors separately.
Enable configurable retries for the bgtask.
* acc forwarder

* version to .0.1
Cleaning out unused imports.
fix(das-api): fix getAssetProof
* fix(ingester): fix compressed nft burn indexing

* Small fix for asssetId calculation
* add dto parameter

* add dao condition

* upd asset_data fetching to ensure single presence of asset_data in query

* use asset.asset_data key for linking between asset/asset_data

* fix: breaking arg order for array queries

* chore: consistent arg order in `SearchAssets` destructuring

* fix: breaking arg count for array params in searchAssets
@tahsintunan tahsintunan changed the title Json uri Merge json-uri optional arg Jun 21, 2023
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we just keep the changes related to json_uri without making any other changes?

@tahsintunan tahsintunan requested review from a user and NicolasPennie June 22, 2023 16:35
@tahsintunan tahsintunan merged commit 25595cb into helius Jun 22, 2023
@tahsintunan tahsintunan deleted the json-uri branch June 22, 2023 16:56
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.

7 participants