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

Premint - rename royaltyRecipient to fundsRecipient on premint v2 signature #289

Merged

Conversation

oveddan
Copy link
Collaborator

@oveddan oveddan commented Oct 20, 2023

Since royalty recipient is both the creator rewarsd recipient and royalty recipient, this PR renames it to
creatorRewardsAndRoyaltyRecipient on the signature, to better imply what it means

@oveddan
Copy link
Collaborator Author

oveddan commented Oct 20, 2023

@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from 6828515 to 642cc5b Compare October 20, 2023 23:32
@oveddan oveddan changed the title Premint - rename royaltyRecipient to fundsRecipient on premint v2 signature Premint - rename royaltyRecipient to creatorRewardsAndRoyaltyRecipient on premint v2 signature Oct 20, 2023
@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from 642cc5b to 81f7546 Compare October 20, 2023 23:33
@oveddan oveddan marked this pull request as ready for review October 20, 2023 23:34
@oveddan oveddan changed the base branch from encoded-bytes-in-sig to fix-create-referral-sig October 21, 2023 00:15
@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from 81f7546 to 4474cad Compare October 21, 2023 00:15
@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from 4474cad to acd8c1e Compare October 23, 2023 17:43
@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from acd8c1e to 575ccbd Compare October 23, 2023 19:58
@oveddan oveddan changed the base branch from fix-create-referral-sig to premint_create_referral October 23, 2023 21:58
@oveddan oveddan changed the base branch from premint_create_referral to premint-better-mint-arguments October 24, 2023 17:12
@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from 575ccbd to 214863d Compare October 24, 2023 17:12
@oveddan oveddan force-pushed the premint-better-mint-arguments branch from 1720bf2 to dcf6b6a Compare October 24, 2023 18:13
@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from 214863d to d193941 Compare October 24, 2023 18:14
@oveddan oveddan force-pushed the premint-better-mint-arguments branch from dcf6b6a to 0012402 Compare October 24, 2023 18:56
@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from d193941 to b873fa9 Compare October 24, 2023 19:02
@oveddan oveddan changed the base branch from default-msg-sender to premint_create_referral November 3, 2023 19:50
@oveddan oveddan force-pushed the rename-royalty-recipient-to-funds-recipient branch from acb98a6 to 8678ee2 Compare November 3, 2023 19:50
@oveddan oveddan merged commit b0bdd61 into premint_create_referral Nov 3, 2023
4 of 5 checks passed
@oveddan oveddan deleted the rename-royalty-recipient-to-funds-recipient branch November 3, 2023 19:50
@oveddan oveddan changed the title Premint - rename royaltyRecipient to creatorRewardsAndRoyaltyRecipient on premint v2 signature Premint - rename royaltyRecipient to fundsRecipient on premint v2 signature Nov 3, 2023
oveddan added a commit that referenced this pull request Nov 6, 2023
oveddan pushed a commit that referenced this pull request Apr 8, 2024
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.


# Releases
## @zoralabs/[email protected]

### Minor Changes

-   13a4785: Adds ERC20 Minter contract which enables NFTs to be purchased with ERC20 tokens

### Patch Changes

-   13a4785: Adds first minter reward to ERC20 Minter
-   1cf02a4: Add ERC7572 ContracTURIUpdated() event for indexing
-   079a596: Moved shared functionality into shared-contracts. premintWithSignerContract takes firstMinter as an argument

## @zoralabs/[email protected]

### Minor Changes

-   13a4785: Adds ERC20Minter contract and events to the subgraph
-   13a4785: Adds ERC20Minter deployment addresses to the subgraph configs

### Patch Changes

-   Updated dependencies [13a4785]
-   Updated dependencies [1cf02a4]
-   Updated dependencies [ec20ac8]
-   Updated dependencies [13a4785]
-   Updated dependencies [079a596]
    -   @zoralabs/[email protected]
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.

1 participant