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

Marketplace API sample calls #943

Merged
merged 13 commits into from
May 8, 2024
Merged

Marketplace API sample calls #943

merged 13 commits into from
May 8, 2024

Conversation

sneakzttv
Copy link
Contributor

@sneakzttv sneakzttv commented May 8, 2024

Marketplace api sample get calls working on sepolia in the sample scene following the api docs here:

https://docs.gaming.chainsafe.io/marketplace-api/docs/marketplaceapi
https://docs.gaming.chainsafe.io/token-api/docs/tokenapi

image

Smaller commits so it's not a large shock later, WIP #912

Additional misc fixes:
DisableGameObjectIfServiceNotActive.cs was returning a non existent dictionary key, added a try get value to make sure it exists first to avoid errors.
Cleaned up old model scripts.

@sneakzttv sneakzttv added the ready-to-merge Ready to Merge PR - this'll trigger required checks label May 8, 2024
@sneakzttv sneakzttv added ready-to-merge Ready to Merge PR - this'll trigger required checks and removed ready-to-merge Ready to Merge PR - this'll trigger required checks labels May 8, 2024
Copy link
Collaborator

@rob1997 rob1997 left a comment

Choose a reason for hiding this comment

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

GJ!

Copy link
Contributor

@juans-chainsafe juans-chainsafe left a comment

Choose a reason for hiding this comment

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

Working all as expected when you set your collectionId, marketplaceId and tokenId!

Great job.


#region Collections

[SerializeField] private string collectionId721 = "d588268b-8a5b-486a-8ea1-4122b510d71e";
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if we can set these values on the editor inspector, so testing is quicker. I also think that should be blank by default, because they will only work for the owner

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we can, ill do this in another pr though

@sneakzttv sneakzttv merged commit 261d33c into dev May 8, 2024
5 checks passed
@sneakzttv sneakzttv deleted the Sneakz/marketplace-scripts branch May 8, 2024 14:07
rob1997 pushed a commit that referenced this pull request Jan 16, 2025
* Marketplace api calls functional

* Cleanup

* Abstraction layer

* Update CSServer.cs

* Updates

* Serialized fields

* Abstraction

* Cleanup

* Update

* Update .gitignore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Ready to Merge PR - this'll trigger required checks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants