status-im/PyGithub
1
0
Fork 0
mirror of https://github.com/status-im/PyGithub.git synced 2026-08-31 10:51:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix: #1671 Convert Python Bool to API Parameter for Authenticated User Notifications (#2001)

Browse Source 
The parameters 'all' and 'participating' for AuthenticatedUser.get_notifications() should
be lower case strings, not textual forms of a boolean.

Fixes: #1671
This commit is contained in:
sshekdar-VMware
2021-10-11 15:44:03 +11:00
committed by GitHub
parent d2d59f9ae8
commit 1da600a343
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
github/AuthenticatedUser.py
+4 -2
View File
@@ -973,9 +973,11 @@ class AuthenticatedUser(github.GithubObject.CompletableGithubObject):
params = dict()
if all is not github.GithubObject.NotSet:
params["all"] = all
# convert True, False to true, false for api parameters
params["all"] = "true" if all else "false"
if participating is not github.GithubObject.NotSet:
params["participating"] = participating
# convert True, False to true, false for api parameters
params["participating"] = "true" if participating else "false"
if since is not github.GithubObject.NotSet:
params["since"] = since.strftime("%Y-%m-%dT%H:%M:%SZ")
if before is not github.GithubObject.NotSet: