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

initial pass at fxhash support #12

Merged
merged 8 commits into from
Jun 25, 2024
Merged

Conversation

yungwkndllc
Copy link
Contributor

@yungwkndllc yungwkndllc commented Jun 21, 2024

This adds support for 2 cases of minting on FxHash -

  • Base chain, on their frame mint contract
  • Mainnet on their fixed price contract

I am sure there are other combinations of types of minting on their stuff out there, but wanted to get started with this at least.

Unit tests included - how do we run end to end testing?

Goes for this bounty - #5

Signed-off-by: yungwknd <yungwkndllc@gmail.com>
Signed-off-by: yungwknd <yungwkndllc@gmail.com>
@yungwkndllc
Copy link
Contributor Author

Added some more tests, and only base support

Signed-off-by: yungwknd <yungwkndllc@gmail.com>
@chrismaddern chrismaddern self-assigned this Jun 22, 2024
Signed-off-by: yungwknd <yungwkndllc@gmail.com>
Signed-off-by: yungwknd <yungwkndllc@gmail.com>
@chrismaddern
Copy link
Contributor

It works!!

You can check out your test mint on Floor here:
http://www.floornfts.io/pdp/pack/5fb27fd7-6ef3-4f17-9dc6-10f88a60b40f

(WIll deep link you on your phone)

Very cool!

I suspect I may have one additional ask later today (we're revving the interface slightly based on early learnings, but should be straightforward). You should be good to go!!

🙏 🥳

@chrismaddern
Copy link
Contributor

FYI — the updates I spoke of our new in an open PR for review by the Floor team 🙏

This hopefully improves your test & run story!! Thanks again for the feedback.

#14

@chrismaddern
Copy link
Contributor

Thanks for your patience @yungwkndllc.

The PR is now merged into the main mobile minting repo. If you rebase against that, and get your ingestor working again, you're good to go!

Thanks for begin first, and thanks for helping improve the framework 🙏 🙏 🙏 🙏

@chrismaddern
Copy link
Contributor

chrismaddern commented Jun 24, 2024

A reminder, you can now try your ingestor out on input with:

yarn dry-run fxhash url https://.......

or...

yarn dry-run fxhash contract 8453:0x.........

@yungwkndllc
Copy link
Contributor Author

A reminder, you can now try your ingestor out on input with:

yarn dry-run fxhash url https://.......

or...

yarn dry-run fxhash contract 8453:0x.........

A reminder, you can now try your ingestor out on input with:

yarn dry-run fxhash url https://.......

or...

yarn dry-run fxhash contract 8453:0x.........

When you ingest contracts for fxhash, are you picking up the minter (sales) contract or the token contract? For example which contract do you pick up for - https://www.fxhash.xyz/generative/slug/allegro

  • Sales - 0x6e625892C739bFD960671Db5544E260757480725
  • Token - 0x914cf2d92b087C9C01a062111392163c3B35B60e

Need to know to implement the new supportsContract and createMintForContract functions..

Signed-off-by: yungwknd <yungwkndllc@gmail.com>
{ internalType: 'uint256', name: '_amount', type: 'uint256' },
{ internalType: 'address', name: '_to', type: 'address' },
],
name: 'buy',
Copy link
Contributor

Choose a reason for hiding this comment

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

For sanity of diff sizes, we favor trimmed ABIs — i.e. including only the objects describing the methods we care about.

In this case, everything except the buy and prices methods can be deleted.

Copy link
Contributor

Choose a reason for hiding this comment

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

I've updated the README based on this too 🙏
ae8208e

@chrismaddern chrismaddern merged commit 29eb609 into floornfts:main Jun 25, 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