homebase-dao-deployer: Optimize by Refactoring Duplicated Code #741
Assignees
Comments
benefacto
changed the title
|
@Man-Jain I understand that the homebase-dao-deployer is hosted on AWS. Is it possible for me to be granted access to it? |
|
WIP pull request: dOrgTech/homebase-dao-deployer#1 |
|
Reconsider, maybe hombase should be moved to Next.js, maybe follow up with Manak |
|
This entire repository should be deprecated by #507 so no need for a refactor |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description:
The off-chain DAO deployer, the homebase-dao-deployer project, currently contains a significant amount of duplicated code. This duplication seems to have originated from adapting the existing homebase-app to create an Express server-based solution for deploying DAOs. This issue aims to track the effort to refactor and optimize the codebase, removing unnecessary duplication and improving overall efficiency.
Background:
The original DAO contracts were too large for direct deployment through the Tezos wallet. As a workaround, an Express API server was developed to handle DAO deployments via JSON requests. During this process, code from
homebase-appwas largely repurposed, leading to redundancy in the codebase.Tasks:
Identify Redundant Code: Review the current codebase to pinpoint exact areas of duplication. This may require a comparison with the original
homebase-appcode.Plan for Refactoring: Develop a clear refactoring strategy that outlines which parts of the code are to be modified, consolidated, or removed.
Implement Code Refactoring: Carry out the planned refactoring, ensuring that no essential functionality is lost or negatively impacted.
Documentation Update: Update any related documentation to reflect changes made during the refactoring process.
Expected Outcome:
The refactored off-chain DAO deployer should be more efficient, with reduced code redundancy, while maintaining all existing functionalities. This should simplify future maintenance and potentially improve deployment performance.
Additional Notes:
homebase-appand the current deployer is advised for better understanding and efficient refactoring.The text was updated successfully, but these errors were encountered: