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

Revenue cli #191

Merged
merged 2 commits into from
Feb 26, 2024
Merged

Revenue cli #191

merged 2 commits into from
Feb 26, 2024

Conversation

JssDWt
Copy link
Collaborator

@JssDWt JssDWt commented Feb 23, 2024

This PR adds a cli for revenue data.
There are 2 commands:

  • export-forwards
  • revenue
NAME:
   lspd_revenue_cli - get revenue data from lspd

USAGE:
   lspd_revenue_cli [global options] command [command options] [arguments...]

VERSION:
   v24.01.14-36-gf093584 commit=f093584bab127fe905831bb28c1b875230a3b9a8

COMMANDS:
   export-forwards  Export forwards with a given peer correlated to an api key for a given time period.
   revenue          Get a revenue report.
   help, h          Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --database-url value  postgres database url for lspd
   --help, -h            show help
   --version, -v         print the version

This PR depends on #189
The data about fees used for channel opens will only be reliable if the above PR was running throughout the entire time period the revenue data is fetched for.

@JssDWt JssDWt requested review from yaslama and hydra-yse February 23, 2024 10:11
@JssDWt JssDWt force-pushed the jssdwt-revenue-cli branch from f093584 to 3abe575 Compare February 23, 2024 10:47
@JssDWt JssDWt requested a review from roeierez February 23, 2024 10:59
Copy link
Member

@hydra-yse hydra-yse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good Jesse! Just those importedIndex changes, and it should be ready.

cmd/lspd_revenue_cli/revenue.go Outdated Show resolved Hide resolved
cmd/lspd_revenue_cli/revenue.go Outdated Show resolved Hide resolved
cmd/lspd_revenue_cli/revenue.go Show resolved Hide resolved
@JssDWt JssDWt force-pushed the jssdwt-revenue-cli branch from 3abe575 to b0e9dfa Compare February 24, 2024 14:16
@JssDWt JssDWt force-pushed the jssdwt-revenue-cli branch from b0e9dfa to ba45715 Compare February 26, 2024 08:33
@JssDWt JssDWt merged commit ba45715 into master Feb 26, 2024
7 checks passed
@JssDWt JssDWt deleted the jssdwt-revenue-cli branch May 24, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants