Skip to content

Refactor: Move Utility Methods Out of Core Business Logic #3568

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.

Sign up for GitHub

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

Jump to bottom
Closed
konstantinabl opened this issue Mar 20, 2025 · 0 comments
Closed

Refactor: Move Utility Methods Out of Core Business Logic #3568

konstantinabl opened this issue Mar 20, 2025 · 0 comments
Assignees
@konstantinabl
Labels
enhancement New feature or request
Milestone
0.68.0

Comments

@konstantinabl
Copy link
Contributor

konstantinabl commented Mar 20, 2025 •
edited
Loading

The EthImpl class (eth.ts) contains many helper methods used by core business logic. These methods should be moved to commonService, allowing newly structured services to depend on commonService rather than directly calling each other.
@konstantinabl konstantinabl moved this from Backlog to Tasks In Progress in Smart Contract Sprint Board Apr 8, 2025
@konstantinabl konstantinabl self-assigned this Apr 8, 2025
@konstantinabl konstantinabl mentioned this issue May 7, 2025
Merged
2 tasks
@quiet-node quiet-node added the enhancement New feature or request label May 7, 2025
@quiet-node quiet-node added this to the 0.68.0 milestone May 7, 2025
@Ferparishuertas Ferparishuertas moved this from Tasks In Progress to In Review in Smart Contract Sprint Board May 9, 2025
@konstantinabl konstantinabl moved this from In Review to Done in Smart Contract Sprint Board May 12, 2025
@konstantinabl konstantinabl closed this as completed by moving to Done in Smart Contract Sprint Board May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@konstantinabl konstantinabl

Labels
enhancement New feature or request
Projects
None yet
Milestone
0.68.0
Development

No branches or pull requests
2 participants
@konstantinabl @quiet-node