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

Index token inscriptions to asset_v1_account_attachments table and add filter showInscription #222

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Index token inscriptions to asset_v1_account_attachments table and add filter showInscription #222

merged 1 commit into from
Jan 7, 2025

Conversation

Nagaprasadvr
Copy link
Collaborator

@Nagaprasadvr Nagaprasadvr commented Nov 19, 2024 •
edited
Loading

  • impl indexing token_inscriptions from libreplex lib
  • save indexed data to asset_v1_account_attachments table inside data field and map asset_id to mint
  • add showInscription logic
  • Add tests

@Nagaprasadvr Nagaprasadvr force-pushed the das-index-token-inscriptions branch from 2e01bac to 53c1cb3 Compare November 19, 2024 11:26
kespinola
kespinola requested changes Nov 27, 2024
View reviewed changes
Copy link
Collaborator

@kespinola kespinola left a comment

Choose a reason for hiding this comment

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

Only minor nits. Thank you.

digital_asset_types/src/dao/scopes/asset.rs Show resolved Hide resolved
digital_asset_types/src/dapi/common/asset.rs Outdated Show resolved Hide resolved
@Nagaprasadvr Nagaprasadvr requested a review from danenbm December 30, 2024 09:04
danenbm
danenbm approved these changes Dec 31, 2024
View reviewed changes
Copy link
Contributor

@danenbm danenbm left a comment

Choose a reason for hiding this comment

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

Approved with question/suggestion

program_transformers/src/lib.rs Show resolved Hide resolved
@Nagaprasadvr @kespinola
Index Token Inscriptions
497bd39 
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.
@kespinola kespinola force-pushed the das-index-token-inscriptions branch from 0313303 to 497bd39 Compare January 7, 2025 11:27
@kespinola kespinola merged commit 313cb5f into metaplex-foundation:main Jan 7, 2025
3 checks passed
@kespinola kespinola deleted the das-index-token-inscriptions branch January 7, 2025 12:39
kespinola pushed a commit to rpcpool/digital-asset-rpc-infrastructure that referenced this pull request Jan 7, 2025
@Nagaprasadvr @kespinola
Index Token Inscriptions (metaplex-foundation#222)
9d8818d 
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 pushed a commit to rpcpool/digital-asset-rpc-infrastructure that referenced this pull request Jan 7, 2025
@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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danenbm danenbm danenbm approved these changes

@kespinola kespinola kespinola approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Nagaprasadvr @kespinola @danenbm