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

Add EIP: Private Equity #7331

Closed
wants to merge 35 commits into from
Closed

Conversation

tokenizeassets
Copy link

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md

We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

  • The PR edits only existing draft PRs.
  • The build passes.
  • Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
  • If matching on email address, the email address is the one publicly listed on your GitHub profile.

@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-erc labels Jul 14, 2023
@eth-bot
Copy link
Collaborator

eth-bot commented Jul 14, 2023

File EIPS/eip-7331.md

Requires 1 more reviewers from @axic, @Pandapip1, @SamWilsn, @xinbenlv

@eth-bot eth-bot changed the title Add EIP: Private-Equity Fund Contract Add EIP: Private Equity Jul 14, 2023
@eth-bot eth-bot added a-review Waiting on author to review e-consensus Waiting on editor consensus labels Jul 14, 2023
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Jul 14, 2023
@eth-bot eth-bot added the e-review Waiting on editor to review label Jul 14, 2023
@github-actions github-actions bot added w-ci Waiting on CI to pass and removed w-ci Waiting on CI to pass labels Jul 14, 2023
@github-actions
Copy link

The commit 49c9fae (as a parent of 8928d25) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Jul 14, 2023
EIPS/eip-7331.md Outdated

## Reference Implementation

[ERC-20](./eip-20.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

why ERC-20 or ERC-3643 is a reference implementation?

Choose a reason for hiding this comment

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

Hi Xin,

The Fund.sol smart contract functionality consists of interacting with ERC-3643 and ERC-20 standard token smart contracts. But in essence, it can interact with any token smart contracts that implements the ERC-20 interface. Do you recommend removing the reference implementations?

Copy link
Contributor

Choose a reason for hiding this comment

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

I mean: the section of "Reference Implementation" is meant for you as author to provide implementations of the standard interface you proposal, in another word, we expect to see a working piece of smart contract code that comply with your proposed ERC

Choose a reason for hiding this comment

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

Hi Xin,

Added the Fund.sol implementation to the section.

Copy link
Member

@lightclient lightclient left a comment

Choose a reason for hiding this comment

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

IMO this doesn't make a lot of sense as an ERC. This seems much more similar to an OpenZepplin library contract. Can you explain what is the advantage of standardizing this?


## Abstract

A private equity fund is a type of investment fund that pools together capital from high-net-worth individuals, institutional investors, and sometimes, pension funds to make investments in private companies. These funds are managed by private equity firms, which specialize in acquiring ownership stakes in companies and actively managing them to generate returns.
Copy link
Member

Choose a reason for hiding this comment

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

This doesn't really explain what this ERC does.

Copy link
Member

Choose a reason for hiding this comment

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

I would say more along the lines of "this standard proposes a standardized interface for Private Equity smart contracts" or something

}
```

## Rationale
Copy link
Member

Choose a reason for hiding this comment

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

missing content here

Copy link
Author

Choose a reason for hiding this comment

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

Hey @lightclient , Updated eip-7331.md with the Rationale.

@mattstam
Copy link
Contributor

Agree with @lightclient that the current design just isn't generic enough to make sense as an ERC.

added rationale
@mattstam
Copy link
Contributor

Thanks for adding Rationale section, definitely helps.

@tokenizeassets
Copy link
Author

Agree with @lightclient that the current design just isn't generic enough to make sense as an ERC.

@mattstam @lightclient, thank you for your comments. Our intention is to provide a modular interface to complement an institution's security token needs for private equity. As this standard matures, it can be used by any institution to offer regulated investment options without major upgrades to the smart contracts.

EIPS/eip-7331.md Outdated
## Specification

```solidity
// SPDX-License-Identifier: GPL-3.0
Copy link
Member

Choose a reason for hiding this comment

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

Must be CC0

EIPS/eip-7331.md Outdated
## Reference Implementation

```solidity
// SPDX-License-Identifier: GPL-3.0
Copy link
Member

Choose a reason for hiding this comment

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

Must be CC0

@@ -0,0 +1,64 @@
// SPDX-License-Identifier: GPL-3.0
Copy link
Member

Choose a reason for hiding this comment

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

Must be CC0

@@ -0,0 +1,159 @@
// SPDX-License-Identifier: GPL-3.0
Copy link
Member

Choose a reason for hiding this comment

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

Must be CC0

@@ -0,0 +1,190 @@
// SPDX-License-Identifier: GPL-3.0
Copy link
Member

Choose a reason for hiding this comment

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

Must be CC0

Choose a reason for hiding this comment

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

Hello Pandapip1,

Thanks, and addressed.

@SamWilsn
Copy link
Contributor

I am closing this pull request because we are in the process of separating EIPs and ERCs into distinct repositories. Unfortunately, as far as we are aware, GitHub does not provide any tools to ease this migration, so every pull request will need to be re-opened manually.

As this is a PR to create / modify an ERC, I will kindly ask you to redirect this to the new repository at ethereum/ERCs. We have prepared a guide to help with the process.

If there is relevant history here, please link to this PR from the new pull request.

On behalf of the EIP Editors, I apologize for this inconvenience.

@SamWilsn SamWilsn closed this Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-new Creates a brand new proposal e-consensus Waiting on editor consensus e-review Waiting on editor to review s-draft This EIP is a Draft t-erc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants