-
Notifications
You must be signed in to change notification settings - Fork 10
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
[PLA-1608] Token metadata fallback, and {id} replacement. #138
Conversation
PR Description updated to latest commit (3d9892c) |
PR Review
Code feedback:
✨ Review tool usage guide:Overview:
With a configuration file, use the following template:
See the review usage page for a comprehensive guide on using this tool. |
PR Code Suggestions
✨ Improve tool usage guide:Overview:
With a configuration file, use the following template:
See the improve usage page for a more comprehensive guide on using this tool. |
Type
enhancement
Description
AttributeType.php
to include bothcollection_chain_id
andtoken_chain_id
.Block.php
.Token.php
for fetching metadata using the collection's URI attribute as a fallback when token's metadata is not directly available.GetTokenTest.php
to assert the new format of URI values that include both collection and token chain IDs.Changes walkthrough
AttributeType.php
Enhanced URI Attribute Resolution with Collection ID Support
src/GraphQL/Types/Substrate/AttributeType.php
with token_chain_id.
Token.php
Enhanced Metadata Fetching with Collection URI Fallback
src/Models/Laravel/Token.php
when token's metadata is not available.
metadata fetching.
Block.php
Documentation Update for Prunable Method Return Type
src/Models/Laravel/Block.php
GetTokenTest.php
Test Update for New URI Value Format
tests/Feature/GraphQL/Queries/GetTokenTest.php
collection_chain_id and token_chain_id.