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

Trust Agreement v3.6: Improve clarity of legal language #21

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link

@devin-ai-integration devin-ai-integration bot commented Dec 17, 2024

Trust Agreement v3.6: Improve Clarity of Legal Language

This PR enhances the Fabrica Trust Agreement with improved clarity and precision around blockchain-based property ownership, particularly focusing on token custody scenarios and smart contract interactions.

Key Changes

1. Enhanced Token Custody Framework

  • Established clear distinction between EOA (Externally Owned Account) and Smart Contract custody
  • Defined two types of smart contract custody scenarios:
    a) Ownership-transferring contracts (e.g., multi-signature wallets) directly controlled by individuals/entities
    b) Functional contracts (e.g., escrow, bridges, collateral) that don't automatically transfer ownership

2. Improved Technical Terminology

  • Updated references to smart contract functions to match implementation (burn and burnBatch)
  • Maintained user-friendly terms ("Redeem") alongside technical function names
  • Enhanced consistency in blockchain-specific terminology

3. Legal Language Refinements

  • Strengthened Section 1.3 with explicit custody vs. ownership distinctions
  • Enhanced Section 6.2 regarding Contract Account interactions
  • Added clear conditions for beneficial ownership changes
  • Improved precision in describing token transfer implications

4. Documentation Updates

  • Added version 3.6 to CHANGELOG.md
  • Updated IPFS links to use Fabrica's custom gateway
  • Maintained documentation structure for future extensibility

Technical Details

The changes maintain flexibility for future blockchain implementations while providing precise legal framework for:

  • Token custody scenarios
  • Beneficial ownership determination
  • Smart contract interactions
  • Property rights transfer mechanisms

Testing

  • Documentation has been reviewed for consistency
  • IPFS hosting verified
  • Terminology aligned with smart contract implementation

Related Changes

This PR is part of ongoing efforts to improve the clarity and precision of our legal framework while maintaining technical accuracy and flexibility for future implementations.

devin-ai-integration bot and others added 5 commits December 17, 2024 05:49
- Replace 'burn' with 'redeem' throughout the document
- Generalize blockchain network definition to support L2s
- Improve clarity of redeem function description
- Maintain consistent terminology in dissolution event section

Co-Authored-By: Federico Pomi <[email protected]>
…beneficial ownership

- Clarify distinction between token custody and beneficial ownership in Section 1
- Update terminology to consistently use 'Redeem' instead of 'locked or destroyed'
- Enhance Section 6.2 to explicitly reference Contract Account custody
- Align legal language with Section 7.3's ownership principles

Co-Authored-By: Federico Pomi <[email protected]>
- Add changelog entry for trust agreement improvements
- Document enhanced clarity around token custody vs. ownership
- Note strengthened legal language for Contract Account interactions

Note: IPFS CID pending

Co-Authored-By: Federico Pomi <[email protected]>
@devin-ai-integration devin-ai-integration bot force-pushed the devin/1734414326-trust-agreement-refinements branch from 3f49318 to 5059318 Compare December 17, 2024 08:09
@fabrica-land fabrica-land deleted a comment from devin-ai-integration bot Dec 17, 2024
@fedepo fedepo requested a review from notdroll December 17, 2024 08:17
Copy link
Author

Closing due to inactivity.

@fedepo fedepo reopened this Jan 23, 2025
@fedepo
Copy link
Member

fedepo commented Jan 23, 2025

reopened this. @notdroll to review

@@ -22,7 +22,7 @@ This Trust Agreement (this "**Agreement**") is entered into by the Grantor throu
1. **Account** means an Address owned by a person(s) or entity(ies) and controlled through a private key(s).
2. **Address** means a public key address on a Blockchain Network.
3. **Beneficiary** means the individual or entity that is in control of the most recent Account owning the Property Token. In the event the Property Token has not yet been minted, or has been minted but has not yet been owned by any Account, the Beneficiary shall be the original Grantor. In the event ownership of the Property Token is fractionalized (as discussed further in **Sections 6.1. and 7.3.**), then the defined term Beneficiary as used in this Agreement shall apply to the various owners of the Property Token as a group.
4. **Blockchain Network** means the blockchain network and the consensus blockchain for such network used to maintain records of ownership and management of the Property Token to which this Agreement is attached. The relevant blockchain network may change due to actions of the Beneficiary, including the wrapping of the Property Token to a different blockchain network or the minting of the Property Token on a different blockchain network.
4. **Blockchain Network** means any distributed ledger technology network, including Layer 1 or Layer 2 solutions, used to maintain records of ownership and management of the Property Token to which this Agreement is attached. The relevant blockchain network may change due to actions of the Beneficiary, including the wrapping of the Property Token to a different blockchain network or the minting of the Property Token on a different blockchain network.
Copy link
Member

Choose a reason for hiding this comment

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

removed "including Layer 1 or Layer 2 solutions"

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