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

[Draft] - Magic Eden API Integration #276

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Tcadle434
Copy link

What changed? Why?

This PR introduces numerous Magic Eden APIs to enable agentkit to interact with NFT trading on Magic Eden. This is in draft and not meant to be merged, as a few additional endpoints still need to be added. Additionally, there are still some design considerations that may change.

Qualified Impact

This shouldn't break any existing functionality as it is an entirely new action provider. If any calls to the ME API should fail, the error will be displayed to the user.

@cb-heimdall
Copy link

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@John-peterson-coinbase John-peterson-coinbase added action provider New action provider triaged needs review PR / issue needs review banana labels Feb 10, 2025
@0xRAG 0xRAG removed the triaged label Feb 10, 2025
@0xRAG
Copy link
Contributor

0xRAG commented Feb 11, 2025

Hey @Tcadle434 anything we can do to support getting your PR ready for review?

@0xRAG 0xRAG added question Further information is requested and removed needs review PR / issue needs review labels Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action provider New action provider banana question Further information is requested
Development

Successfully merging this pull request may close these issues.

4 participants