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

Ingestor MagicEden #33

Closed
wants to merge 4 commits into from
Closed

Conversation

pbkompasz
Copy link
Contributor

Mint Ingestor: MagicEden(#11)

Functionality Supported

  • Ingesting from URL: Yes / No
  • Ingesting from Contract address: Yes / No
  • Supported Networks: Base / Ethereum / Arbitrum / Zora / Polygon / Others..

Before you submit

  • Ensure your generated MintTemplate works 😄
  • Ensure that your code is restricted to a single folder in src/ingestors
  • Ensure that all required assets are included (e.g. ABIs)
  • Ensure ABIs are trimmed to include only methods (1) used in the ingestor or (2) required to mint
  • Ensure that all exported methods are prefixed with the name of your ingestor e.g. myMintingPlatformGetContractDetails
  • Ensure that a test exists for generating a MintTemplate that will always succeed
  • Ensure that your code accesses no external resources except those passed in the resources object

@pbkompasz pbkompasz changed the title Ingestor magiceden Ingestor MagicEden Jul 30, 2024
@chrismaddern chrismaddern self-requested a review August 26, 2024 20:28
@chrismaddern
Copy link
Contributor

@pbkompasz this is still in Draft & has conflicts with main now.

If its' ready for review, can you mark as ready and get the merge conflicts resolved. I'm putting aside time this week to close out all of these! :)

@pbkompasz
Copy link
Contributor Author

@chrismaddern It doesn't work anymore. The api endpoints are blocked for non-browser requests. I am closing it.

@pbkompasz pbkompasz closed this Aug 28, 2024
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.

2 participants