Skip to content

Commit 8414be7

Browse files
Merge pull request #210 from supertokens/fix/syncio-optional-user-context
fix: Make user_context optional in userroles syncio functions
2 parents eba058b + 641a93c commit 8414be7

File tree

2 files changed

+16
-9
lines changed

2 files changed

+16
-9
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
66

77
## [unreleased]
88

9+
### Bug fix
10+
- Make `user_context` optional in userroles recipe syncio functions.
11+
912
## [0.10.1] - 2022-07-11
1013

1114
### Documentation:

supertokens_python/recipe/userroles/syncio/__init__.py

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,31 +39,31 @@ def add_role_to_user(
3939

4040

4141
def remove_user_role(
42-
user_id: str, role: str, user_context: Dict[str, Any]
42+
user_id: str, role: str, user_context: Union[Dict[str, Any], None] = None
4343
) -> Union[RemoveUserRoleOkResult, UnknownRoleError]:
4444
from supertokens_python.recipe.userroles.asyncio import remove_user_role
4545

4646
return sync(remove_user_role(user_id, role, user_context))
4747

4848

4949
def get_roles_for_user(
50-
user_id: str, user_context: Dict[str, Any]
50+
user_id: str, user_context: Union[Dict[str, Any], None] = None
5151
) -> GetRolesForUserOkResult:
5252
from supertokens_python.recipe.userroles.asyncio import get_roles_for_user
5353

5454
return sync(get_roles_for_user(user_id, user_context))
5555

5656

5757
def get_users_that_have_role(
58-
role: str, user_context: Dict[str, Any]
58+
role: str, user_context: Union[Dict[str, Any], None] = None
5959
) -> Union[GetUsersThatHaveRoleOkResult, UnknownRoleError]:
6060
from supertokens_python.recipe.userroles.asyncio import get_users_that_have_role
6161

6262
return sync(get_users_that_have_role(role, user_context))
6363

6464

6565
def create_new_role_or_add_permissions(
66-
role: str, permissions: List[str], user_context: Dict[str, Any]
66+
role: str, permissions: List[str], user_context: Union[Dict[str, Any], None] = None
6767
) -> CreateNewRoleOrAddPermissionsOkResult:
6868
from supertokens_python.recipe.userroles.asyncio import (
6969
create_new_role_or_add_permissions,
@@ -73,23 +73,23 @@ def create_new_role_or_add_permissions(
7373

7474

7575
def get_permissions_for_role(
76-
role: str, user_context: Dict[str, Any]
76+
role: str, user_context: Union[Dict[str, Any], None] = None
7777
) -> Union[GetPermissionsForRoleOkResult, UnknownRoleError]:
7878
from supertokens_python.recipe.userroles.asyncio import get_permissions_for_role
7979

8080
return sync(get_permissions_for_role(role, user_context))
8181

8282

8383
def remove_permissions_from_role(
84-
role: str, permissions: List[str], user_context: Dict[str, Any]
84+
role: str, permissions: List[str], user_context: Union[Dict[str, Any], None] = None
8585
) -> Union[RemovePermissionsFromRoleOkResult, UnknownRoleError]:
8686
from supertokens_python.recipe.userroles.asyncio import remove_permissions_from_role
8787

8888
return sync(remove_permissions_from_role(role, permissions, user_context))
8989

9090

9191
def get_roles_that_have_permission(
92-
permission: str, user_context: Dict[str, Any]
92+
permission: str, user_context: Union[Dict[str, Any], None] = None
9393
) -> GetRolesThatHavePermissionOkResult:
9494
from supertokens_python.recipe.userroles.asyncio import (
9595
get_roles_that_have_permission,
@@ -98,13 +98,17 @@ def get_roles_that_have_permission(
9898
return sync(get_roles_that_have_permission(permission, user_context))
9999

100100

101-
def delete_role(role: str, user_context: Dict[str, Any]) -> DeleteRoleOkResult:
101+
def delete_role(
102+
role: str, user_context: Union[Dict[str, Any], None] = None
103+
) -> DeleteRoleOkResult:
102104
from supertokens_python.recipe.userroles.asyncio import delete_role
103105

104106
return sync(delete_role(role, user_context))
105107

106108

107-
def get_all_roles(user_context: Dict[str, Any]) -> GetAllRolesOkResult:
109+
def get_all_roles(
110+
user_context: Union[Dict[str, Any], None] = None
111+
) -> GetAllRolesOkResult:
108112
from supertokens_python.recipe.userroles.asyncio import get_all_roles
109113

110114
return sync(get_all_roles(user_context))

0 commit comments

Comments
 (0)