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.

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

Added Token Type Filter in search_assets fuction #186

Closed
wants to merge 16 commits into from
Closed

Added Token Type Filter in search_assets fuction #186

wants to merge 16 commits into from

Conversation

AhzamAkhtar
Copy link
Collaborator

@AhzamAkhtar AhzamAkhtar commented Dec 18, 2024 •
edited
Loading

  1. Types of filters that are added in search_assets function are
  • Fungible
  • NonFungible
  • Compressed
  • Nft
  • All
  1. Search_asset rpc call skipping fungible_assets bug fixed
  2. Not able to search for fungible asset using specific owner_address in search_asset bug fixed
  3. Added validation errors and fixed search_assets compatibility issues
  • owner_type and token_type parameter can't be use together
  • In search_assets, SearchAssetQuery doesn't support interface fixed.
  • token_type filter should only be used using owner_address
  • token_type filter can't be used with interface
  1. Tests

@AhzamAkhtar AhzamAkhtar changed the title Das 51 search asset token type filter Added Token Type Filter in search_assets fuction Dec 18, 2024
@AhzamAkhtar AhzamAkhtar marked this pull request as draft December 23, 2024 12:12
@AhzamAkhtar AhzamAkhtar force-pushed the das-51-search-asset-token-type-filter branch 2 times, most recently from bfe1fe0 to e1ad545 Compare December 27, 2024 11:42
@AhzamAkhtar AhzamAkhtar marked this pull request as ready for review December 27, 2024 13:43
@kespinola kespinola force-pushed the das-38-rebase-stale-te-branch branch 3 times, most recently from ccd7925 to ded0fac Compare January 6, 2025 10:18
@AhzamAkhtar AhzamAkhtar force-pushed the das-51-search-asset-token-type-filter branch 2 times, most recently from 643ba3a to 3aba596 Compare January 7, 2025 16:46
Nagaprasadvr and others added 16 commits January 7, 2025 22:19
@Nagaprasadvr @AhzamAkhtar
add remaining extensions (metaplex-foundation#225)
6150b6b
@Nagaprasadvr @AhzamAkhtar
Query NFT Editions (metaplex-foundation#216)
78b0fa4 
Associate editions to the nft mint of the master edition. Add new method
to API get_nft_editions to find editions for the mint.
@Nagaprasadvr @AhzamAkhtar
Index Token Inscriptions (metaplex-foundation#222)
dcabf79 
Program parser for inscriptions for blockbluster. Create and register
program parser with program tranformer. Add show_instrcitions flag to
the API to view the inscription information.
@AhzamAkhtar
Search asset token type filter added
1383ada
@AhzamAkhtar
Token type class filter improved
975e3ed
@AhzamAkhtar
removed specification_asset_class from SearchAssetQuery cause we now …
01e361e 
…have token_type
@AhzamAkhtar
test file added for token_type_filter
69b6cd1
@AhzamAkhtar
tests snapshots added
1e053c4
@AhzamAkhtar
fix tests and added snapshot for search_assets with token_type fungible
86db31e
@AhzamAkhtar
search_assets rpc call skipping fungible_tokens bug resolved
4a02cc9
@AhzamAkhtar
not able to search for fungible asset for owner_address in search_ass…
8fd296a 
…ets fun bug fixed
@AhzamAkhtar
fixed search asset compatibility issues
deda9ef
@AhzamAkhtar
owner_type and token_typenot be used together feature added and test …
18751f3 
…improved
@AhzamAkhtar
fungible assets not showing when token_type is set to all bug fixed
8975599
@AhzamAkhtar
cleanup
fa3eb55
@AhzamAkhtar
conflict resolved
d8e4234
@AhzamAkhtar AhzamAkhtar force-pushed the das-51-search-asset-token-type-filter branch from 3aba596 to d8e4234 Compare January 7, 2025 16:50
@AhzamAkhtar AhzamAkhtar closed this Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AhzamAkhtar @Nagaprasadvr