Limited to 30 gists? about nppgist HOT 14 CLOSED

I'm going to implement pagination ASAP.

from nppgist.

Could you please check the latest version https://ci.appveyor.com/project/KvanTTT/nppgist/builds/35074337 on your win 7 system?

from nppgist.

Works like a charm! Thanks for your patience!

My system is running .net client/full: 4.7.02558

from nppgist.

I don't think so, it's quite strange.

from nppgist.

Correction, it is showing 30, not 29 as I initially wrote.

I have a script that lists my gists and it has the same issue, only showing 30 of 52 gists,
Digging further I found out the API changed a bit.:
https://docs.github.com/en/rest/reference/gists#list-gists-for-a-user
https://developer.github.com/v3/#pagination

<authenticated session> 
curl https://api.github.com/users/{username}/gists | jq-win64 ".[]|.files[].filename" gistout.json | wc -l
30

<authenticated session>
curl "https://api.github.com/users/{username}/gists?page=1&per_page=100" | jq-win64 ".[]|.files[].filename" gistout.json | wc -l
52

I haven't read through the source code, but can this be the issue?

BR Agail

from nppgist.

Ideally, we should implement pagination. But a more simple solution - just to enlarge the number of taken gists (but max 100).

from nppgist.

I've implemented it, check in the latest build: https://ci.appveyor.com/project/KvanTTT/nppgist/builds/35063667 Also, you can configure GistsPerPage parameter (default 30) in ini settings file.

from nppgist.

Running npp 7.8.9 x64

Failed to load

NppGist.dll is not compatible with the current version of Notepad++.

Do you want to remove this plugin from the plugins directory to prevent this message from the next launch?

Yes, I picked the x64 version of the dll ;)

from nppgist.

It works fine on my computer (1.4.0.24). Could you please check the x86 version?

from nppgist.

Same issue with npp 7.8.9 x86

Also, I found this in my inbox from github

On September 5th, 2020 at 20:44 (UTC) your personal access token (notepad++) using NppGist was used as part of a query parameter to access an endpoint through the GitHub API:

https://api.github.com/user

Please use the Authorization HTTP header instead, as using the `access_token` query parameter is deprecated. If this token is being used by an app you don't have control over, be aware that it may stop working as a result of this deprecation.

Depending on your API usage, we'll be sending you this email reminder on a monthly basis for each token and User-Agent used in API calls made on your behalf.
Just one URL that was accessed with a token and User-Agent combination will be listed in the email reminder, not all.

Visit https://developer.github.com/changes/2020-02-10-deprecating-auth-through-query-param for more information about suggested workarounds and removal dates.

Thanks,
The GitHub Team

from nppgist.

Such an issue is definitely resolved, I and another user have checked it, see #12

from nppgist.

Good to know, I missed that, sorry.
I’ll try your .23 release when I get back to see if that also trigger the error message.
Stay tuned ..

from nppgist.

Success, well sort of anyway ;)

I went ahead and tried npp 7.8.8 and 7.8.9 with nppGist 1.4.0.23 and 1.4.0.24, both x86 and x64.
Still same issue, then I realized it might be my win7 🤦
Works as expected on my win10 👌

Is win7 supported at all or am I missing some dependencies perhaps? (yeah, I know its no longer supported by MS, but still.)

from nppgist.

Maybe it's something related to the .NET framework version? What version do you have installed on Win 7? NppGist requires .NET 4.7.2 since 1.3.0 (01.12.2019).

Wait, I'll try to downgrade .NET version down to 4.6.1.

from nppgist.