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

feat: Add plugin-nft-generation: create Solana NFT collections. #1011

Merged
merged 6 commits into from
Dec 14, 2024

Conversation

xwxtwd
Copy link
Contributor

@xwxtwd xwxtwd commented Dec 12, 2024

Add plugin-nft-generation: support for creating Solana NFT collections.

Relates to:

N/A

Risks

Low - This plugin addition is isolated and doesn't affect existing functionality.

Background

What does this PR do?

The plugin supports creating Solana NFT collections through API integration and client interaction

What kind of change is this?

Features (non-breaking change which adds functionality)

Documentation changes needed?

Not required (environment variables are commented in .env.example)

Testing

Where should a reviewer start?

  1. Review the plugin installation process
  2. Check the NFT generation functionality
  3. Verify API integration implementation

Detailed testing steps

  1. Configure the following environment variables:
# Solana
SOLANA_PRIVATE_KEY=
SOLANA_PUBLIC_KEY=
SOLANA_ADMIN_PRIVATE_KEY= # This wallet is used to verify NFTs
SOLANA_ADMIN_PUBLIC_KEY= # This wallet is used to verify NFTs
SOLANA_VERIFY_TOKEN= # Authentication token for calling the verification API
  1. Start the agent using any client (direct, Twitter, Telegram, etc.)
  2. Tell the agent the instructions for creating a collection

@xwxtwd xwxtwd changed the base branch from main to develop December 12, 2024 10:51
@blockiosaurus
Copy link

Would it make sense to target the newer Core standard so minting costs are lower? Happy to help with that if you think it makes sense.

@xwxtwd
Copy link
Contributor Author

xwxtwd commented Dec 12, 2024

Would it make sense to target the newer Core standard so minting costs are lower? Happy to help with that if you think it makes sense.

Yes, that makes sense. Thanks for offering to help!

@cygaar cygaar added the Plugin_new Mark PRs that are a new plugin label Dec 12, 2024
@odilitime
Copy link
Collaborator

not sure I like it having it's own webserver, seems to duplicate a lot of what direct-client offers in that regard but since direct-client isn't extensible yet, we'll allow it for now. But will likely need to be change this in the future. And convert these into actions agents can take.

@xwxtwd
Copy link
Contributor Author

xwxtwd commented Dec 13, 2024

1 failing check because the secrets.OPENAI_API_KEY configuration is missing.

#1027

@xwxtwd xwxtwd closed this Dec 13, 2024
@xwxtwd xwxtwd reopened this Dec 13, 2024
agent/src/index.ts Outdated Show resolved Hide resolved
@odilitime odilitime deleted the branch elizaOS:develop December 13, 2024 02:37
@odilitime odilitime closed this Dec 13, 2024
@odilitime odilitime reopened this Dec 13, 2024
@odilitime odilitime deleted the branch elizaOS:develop December 13, 2024 19:07
@odilitime odilitime closed this Dec 13, 2024
@odilitime odilitime reopened this Dec 13, 2024
# Conflicts:
#	agent/src/index.ts
#	packages/core/src/types.ts
#	pnpm-lock.yaml
@lalalune lalalune merged commit 0ca14a3 into elizaOS:develop Dec 14, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Plugin_new Mark PRs that are a new plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants