Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

log balance along with liabilities so we don't need to look at horizon for this data separately #38

Closed
nikhilsaraf opened this issue Oct 29, 2018 · 0 comments
Assignees
Labels
feature request New feature or request
Milestone

Comments

@nikhilsaraf
Copy link
Contributor

Desired Behavior

I want to be able to debug IEIF/CAP-3 issues just by looking at log files, which should contain asset balance data when logging out liability data.

Impact

The desired behavior will allow me to do so since the data needed from horizon would be included in the log.

Feature Suggestion

We can achieve the desired behavior by fetching the horizon balance anytime we log liability data and including that in the log line.

We should also consider including the liability information that we get from horizon so we can compare in case there are any errors in our liability calculations.

@nikhilsaraf nikhilsaraf added the feature request New feature or request label Oct 29, 2018
@nikhilsaraf nikhilsaraf added this to the v1.2.0 milestone Oct 29, 2018
@nikhilsaraf nikhilsaraf self-assigned this Oct 29, 2018
@nikhilsaraf nikhilsaraf modified the milestones: v1.2.0, v1.1.2 Oct 30, 2018
nikhilsaraf added a commit to nikhilsaraf/kelp that referenced this issue Oct 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature request New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant