search

macxred / pyledger

Python package to streamline the implementation and management of accounting systems.
MIT License
0 stars 0 forks source link

Move the `txn_to_str()` method to the `LedgerEngine` class #21

Closed AlexTheWizardL closed 2 months ago

AlexTheWizardL commented 2 months ago

Task:

The txn_to_str() method is currently duplicated in multiple places, but it should exist in only one location. To resolve this:

  1. Move the method to the LedgerEngine class, ensuring that it can be inherited properly by all relevant classes.
  2. After relocating the method, write comprehensive tests to verify its functionality across the entire class hierarchy.
  3. Update all instances where the method is used to follow the new approach, ensuring consistency throughout the codebase.