Skip to content

Conversation

@bhavidhingra
Copy link
Contributor

@bhavidhingra bhavidhingra commented Apr 1, 2025

This PR introduces a generic way of defining NFT collections. It will be used to onboard Aptos NFTs.
The aim is to support specific NFT collection IDs.

TICKET: COIN-3613

@bhavidhingra bhavidhingra requested a review from a team April 1, 2025 14:14
@bhavidhingra bhavidhingra requested review from a team as code owners April 1, 2025 14:14
@bhavidhingra bhavidhingra force-pushed the coin-3613-generic-nft-statics branch from 15e6edb to 9889624 Compare April 1, 2025 14:17
Copy link
Contributor

@baltiyal baltiyal left a comment

Choose a reason for hiding this comment

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

Test cases are failing. Please fix it.

@bhavidhingra bhavidhingra marked this pull request as draft April 2, 2025 05:28
@bhavidhingra bhavidhingra force-pushed the coin-3613-generic-nft-statics branch from 9889624 to 0ff9090 Compare April 2, 2025 05:51
@bhavidhingra bhavidhingra dismissed baltiyal’s stale review April 2, 2025 06:36

fixed the test cases

@bhavidhingra bhavidhingra requested a review from baltiyal April 2, 2025 06:36
@bhavidhingra bhavidhingra marked this pull request as ready for review April 2, 2025 06:37
@bhavidhingra bhavidhingra force-pushed the coin-3613-generic-nft-statics branch from 0ff9090 to 4a3affe Compare April 2, 2025 10:18
@bhavidhingra bhavidhingra force-pushed the coin-3613-generic-nft-statics branch from 4a3affe to 519935b Compare April 2, 2025 10:28
@bhavidhingra bhavidhingra requested a review from baltiyal April 2, 2025 10:29
Copy link
Contributor

@baltiyal baltiyal left a comment

Choose a reason for hiding this comment

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

LGTM

@bhavidhingra bhavidhingra merged commit 01b619a into master Apr 2, 2025
11 checks passed
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.

3 participants