nixnote2 icon indicating copy to clipboard operation
nixnote2 copied to clipboard

Published 20 hours ago • verified publisher icon robert7

Low down the occurences of the rate limit exceptions.

Open boo-yee opened this issue 9 months ago • 2 comments

boo-yee avatar May 01 '24 14:05 boo-yee

Hi @boo-yee, thanks, I'll check then changes.

robert7 avatar May 02 '24 06:05 robert7

Sorry but I have not completed yet, maybe have to work for more time. :(

boo-yee avatar May 02 '24 07:05 boo-yee

I updated the post, adding some details, and I think this pr is ready for reviewing now.

boo-yee avatar May 02 '24 14:05 boo-yee

Hi, I can't test the changes myself now, but it looks good, and I merged your changes to develop.

Unfortunately Travis build doesn't work now, as my free account expired, but I already wrote to Travis, hopefully they resume my account. Robert

robert7 avatar May 07 '24 05:05 robert7

That's all right. We can still do it by ourselves by hand. Thanks for your work.

boo-yee avatar May 07 '24 06:05 boo-yee

There is a parameter of getNote function called withResource. If it is set to true, the api caller will download the reource along with the note. that is why I think the commented out code is tedious.

boo-yee avatar May 07 '24 06:05 boo-yee

It seems that the Travis CI works now and the latest AppImage has been built. Thanks again.

boo-yee avatar May 07 '24 11:05 boo-yee

Travis account is renewed and should work until end of 2024 (then it would need another renew)

robert7 avatar May 07 '24 12:05 robert7