Skip to content

Add initial Tidal contract & supporting scripts & transactions#1

Merged
sisyphusSmiling merged 2 commits intomainfrom
gio/add-tidal-contract
May 22, 2025
Merged

Add initial Tidal contract & supporting scripts & transactions#1
sisyphusSmiling merged 2 commits intomainfrom
gio/add-tidal-contract

Conversation

@sisyphusSmiling
Copy link
Contributor

@sisyphusSmiling sisyphusSmiling commented May 14, 2025

Description

  • Adds initial Tidal contract framing out Tide and TideManager resources
  • Adds supporting transactions and scripts enabling minimal configuration, deletion, interaction with and querying of Tide objects
  • Note that the contents of the Tide resource will change once yield bearing strategies can be constructed from DeFiBlocks components. The initial version is included for framing out top-level abstraction for the purpose of front & backend mocking and composing platform transactions.

@sisyphusSmiling sisyphusSmiling marked this pull request as ready for review May 15, 2025 20:45
@sisyphusSmiling sisyphusSmiling self-assigned this May 15, 2025
access(all) let TideManagerStoragePath: StoragePath
access(all) let TideManagerPublicPath: PublicPath

access(all) event CreatedTide(id: UInt64, initialAmount: UFix64, creator: Address?)
Copy link
Member

Choose a reason for hiding this comment

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

Maybe could include UUID here? Same with the other events

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in 77d04ad

}
}

access(all) resource Tide : Burner.Burnable, FungibleToken.Receiver, FungibleToken.Provider, ViewResolver.Resolver {
Copy link
Member

Choose a reason for hiding this comment

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

I know it is cliche, but just wanted to float it, but would it make sense to make this an NFT? This looks a lot like an NFT and TideManager looks a lot like a NFT collection

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Definitely very similar, but I've been told we want to avoid this. I think the concern is users may transfer Tides without realizing that the ownership of the Tide is inherent to the ownership of the NFT. Other protocols have implemented DeFi positions as NFTs and there have been instances of users selling their positions for far less than they're worth because they think the NFT is just a sort of commemorative token and not the thing itself.

@sisyphusSmiling sisyphusSmiling merged commit 3e62506 into main May 22, 2025
@sisyphusSmiling sisyphusSmiling deleted the gio/add-tidal-contract branch May 22, 2025 18:45
@jribbink jribbink mentioned this pull request Feb 27, 2026
6 tasks
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