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

Inconsistent sorting between Payments and Invoices pages #1295

Closed
ok300 opened this issue Oct 1, 2023 · 1 comment · Fixed by #1300
Closed

Inconsistent sorting between Payments and Invoices pages #1295

ok300 opened this issue Oct 1, 2023 · 1 comment · Fixed by #1300
Assignees
Labels
bug Something isn't working LND

Comments

@ok300
Copy link

ok300 commented Oct 1, 2023

Describe the bug

On the Invoices page, the default sorting direction is Descending. Indeed, the top-most rows in the table are the most recent invoices.

On the Payments page, the default sorting direction is also Descending. However, here the top-most entries are the oldest payments on the node.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Transactions menu > Invoices
  2. Select "Invoices per page" to be low enough that you have your table split across multiple pages
  3. Go to the first page of the table
  4. Notice that the top-most invoice shown is the most recent one on the node
  5. Go to Transactions menu > Payments
  6. Select "Payments per page" to be low enough that you have your table split across several pages
  7. Go to the first page of the table
  8. Notice that the top-most payment shown is the oldest one on the node

Your environment

  • Version of RTL: v0.14.0-beta
  • Version of lnd/core lightning/eclair: LND v0.16.4-beta
  • Version of btcd, bitcoind, or other backend: bitcoind
@ShahanaFarooqui ShahanaFarooqui added this to the Release 0.14.1-beta milestone Oct 4, 2023
@ShahanaFarooqui ShahanaFarooqui self-assigned this Oct 4, 2023
@ShahanaFarooqui ShahanaFarooqui added bug Something isn't working LND labels Oct 4, 2023
ShahanaFarooqui added a commit that referenced this issue Oct 4, 2023
@ShahanaFarooqui ShahanaFarooqui linked a pull request Oct 4, 2023 that will close this issue
@ShahanaFarooqui
Copy link
Collaborator

@ok300 Thank you for raising the issue. It has been fixed with PR #1300 and will be released with version 0.14.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working LND
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants