search

dlvhdr / gh-dash

A beautiful CLI dashboard for GitHub 🚀
https://dlvhdr.github.io/gh-dash
MIT License
7.06k stars 208 forks source link

[BUG] GraphQL: This query requests up to 506,650 possible nodes which exceeds the maximum limit of 500,000. #342

Closed sdwheeler closed 6 months ago

sdwheeler commented 7 months ago

Describe the bug Receiving an error at startup. Also, the error display causes the layout in the terminal to be corrupted.

The log seems to show that the error occurs when querying the PRs for the MicrosoftDocs/PowerShell-Docs-DSC repo. That repo only has a total of 246 issues and PRs.

To Reproduce Start dash with my configuration. The problem is intermittent.

Screenshots I will add a screenshot when the error occurs again.

Desktop (please complete the following information):

sdwheeler commented 7 months ago

Here is a screenshot showing the error and how it breaks the layout in the terminal. image

Interesting to note, I only see this error on screen when I run dash without the --debug switch. If I use --debug, the error is in the error log but not on screen.

KyleKing commented 6 months ago

I'm also having the same issues with both v3.14.0 and HEAD, but not when downgrading to v3.13.1:

gh extension install dlvhdr/gh-dash --pin='v3.13.1' --force
dlvhdr commented 6 months ago

You can use HEAD until I release a new version, #348 should fix the problem. For anyone that doesn't want to use HEAD, you can set a lower limit in your config:

prSections:
  - title: My PRs
    layout:
      lines:
        hidden: true
    filters: author:@me
    limit: 5
KyleKing commented 6 months ago

Thanks!!

On the workaround, shouldn't default.prsLimit be used when set? https://github.com/KyleKing/dotfiles/blob/67371a68ccc550a1b336bd4d6f1484ee1ee7841d/dot_config/gh-dash/config.yml#L33

dlvhdr commented 6 months ago

Yeah it should fallback to that of the section doesn't define one. I guess even with 50 prs, I fetch too many nodes (a pr includes comments, review threads and a whole lot more nodes)