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

refactor: Standardize ABI Handling Across the dApp to Avoid Internal Folder Dependencies #160

Open
3 tasks
eduramme opened this issue Aug 23, 2024 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@eduramme
Copy link
Collaborator

eduramme commented Aug 23, 2024

Refactor Request

Description

There are currently multiple methods being used to fulfill the ABI parameter within the dApp. This inconsistency can lead to maintainability issues and unnecessary complexities. To streamline development and reduce dependencies, it's suggested to adopt a single standard method for handling ABIs across the entire application. The method used in the referenced code avoids bringing ABIs into internal folders and should be applied universally.

Requirements

  • Review all instances where the ABI parameter is being handled differently across the dApp.
  • Standardize the ABI handling by adopting the method used in the referenced code, which minimizes the need for storing ABIs in internal folders.
  • Refactor existing implementations to align with this new standard.

Additional Information

  • This standardization will simplify the codebase and reduce the risk of inconsistencies or errors related to ABI management.
  • Ensure that the transition to the new standard is well-documented and that all relevant team members are informed of the change.

Image

Possible Solution

Audit the current ABI handling methods across the project and refactor them to use the approach demonstrated in the provided code snippet.

@eduramme eduramme added the enhancement New feature or request label Aug 23, 2024
@alextnetto alextnetto added this to the Nice to haves milestone Sep 9, 2024
@eduramme eduramme removed their assignment Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 💡 Ideas
Development

No branches or pull requests

2 participants