inga-lovinde
/
test-assignment-payments
1
0
Fork 0
Code Issues 8 Pull Requests Packages Projects Releases Wiki Activity
Labels Milestones
New Issue

Refactor transaction controller for simplicity #10

Closed
opened 4 months ago by inga-lovinde · 0 comments
inga-lovinde commented 4 months ago
Owner

There is a code duplication between getAmountWithExchange and getAmountWithoutExchange which is no longer actually needed, and also I don't like the way how applyRules is implemented. This is a good candidate to clean things up somewhat.

There is a code duplication between `getAmountWithExchange` and `getAmountWithoutExchange` which is no longer actually needed, and also I don't like the way how `applyRules` is implemented. This is a good candidate to clean things up somewhat.
inga-lovinde closed this issue 4 months ago
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
interview
main
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: inga-lovinde/test-assignment-payments#10
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(<nil>)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes