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

Add createCollection API to the SDK #87

Merged
merged 2 commits into from
Oct 18, 2023
Merged

Conversation

heliuchuan
Copy link
Contributor

Description

This adds a function for users to invoke 0x4::aptos_token::create_collection via the SDK

Test Plan

pnpm test

Related Links

Copy link
Collaborator

@0xmaayan 0xmaayan left a comment

Choose a reason for hiding this comment

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

this should be under digitalAsset class as all aptos_token operations. we can skip it for now as this is a small implementation of a bigger item.
Or at least have this api to be under digitalAsset

@gregnazario
Copy link
Collaborator

this should be under digitalAsset class as all aptos_token operations. we can skip it for now as this is a small implementation of a bigger item. Or at least have this api to be under digitalAsset

Yes, let's combine collection and token under digitalAsset

@gregnazario gregnazario force-pushed the add-create-collection branch from bdf527c to 88d8c60 Compare October 18, 2023 03:11
@gregnazario
Copy link
Collaborator

@heliuchuan just for reference, I addressed the merge conflicts, we can get these two PRs in, and then combine them into one digitalAssets, which should just be an easy rename.

tests/e2e/api/collection.test.ts Outdated Show resolved Hide resolved
Comment on lines +57 to +61
} & CreateCollectionOptions,
): Promise<SingleSignerTransaction> {
return createCollectionTransaction({ aptosConfig: this.config, ...args });
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

Returning a SingleSigner may be weird if we also support Fee payer, but I'm not sure how that model is going to look cc @0xmaayan

@gregnazario gregnazario force-pushed the add-create-collection branch from 88d8c60 to ff50886 Compare October 18, 2023 03:20
@gregnazario gregnazario dismissed 0xmaayan’s stale review October 18, 2023 03:23

Will address in future PR

@gregnazario gregnazario merged commit 078acc3 into main Oct 18, 2023
5 checks passed
@gregnazario gregnazario deleted the add-create-collection branch October 18, 2023 03:23
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