-
Notifications
You must be signed in to change notification settings - Fork 158
Create: DNFactory #64
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
base: main
Are you sure you want to change the base?
Conversation
|
Checks failed because solc version is 0.8.4 vs the contracts had 0.8.24 @QuitCrypto |
649c138 to
f4a6746
Compare
f4a6746 to
638c510
Compare
|
my opinion: |
|
@QuitCrypto I suggest if we could have event like DN404Created emitted would be great to track tokens on subgraph, etc. |
@codebuster22 Where this factory lives isn't SUPER important. But the reasoning behind having a factory contract is it will be tied to a dapp where people can easily deploy verified DN404 implementations (there will likely be more than just the Cloneable example included here). A lot of people that may want to deploy tokens aren't on the most technical side, so giving them an easy way to approach the deployment of the contract is important. Yes, this does increase the chances of more projects that may not succeed, but the pros outweigh that here to be honest in order to give people easy access to safe and verified DN404 implementations. The goals for this is so you can easily fact check if a contract you're about to interact with is truly using unmodified DN404 base smart contracts, since there are so many fake ones out there. |
@pop-punk I believe the DN404 repository should primarily serve as a library, focusing on the core DN404 code. This approach will keep the repository streamlined and dedicated to standard implementations and examples, avoiding the complexities associated with deployment management. Managing deployment scripts, factory contract addresses, and various factory versions within the DN404 repository shifts its purpose from a library to more of a project management hub, potentially complicating management and deterring community contributions. My Suggestion:
Benefits of This Approach:
I believe adopting this structured approach will not only foster greater community involvement but also ensure the integrity of the DN404 core code, all while providing a clear framework for the development and application of DN404 standards. Though the final decision lies with the core contributors and you. Anyways, I'm always happy to contribute. |
|
@codebuster22 Awesome-DN404 sounds good, just like https://github.com/mudgen/awesome-diamonds. |
Description
Creates a DN Factory for easy deployments of DN404 Clones. Includes ability to customize name and symbol, and lock liquidity for an arbitrary number of seconds from deploy.
Checklist
Ensure you completed all of the steps below before submitting your pull request:
forge fmt?forge snapshot?forge test?