Skip to content

Commit

Permalink
Fix long lines
Browse files Browse the repository at this point in the history
  • Loading branch information
bhavberi committed Jun 11, 2024
1 parent d8a1347 commit c4fc3ad
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 8 deletions.
5 changes: 3 additions & 2 deletions models.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ def transform_uid(cls, v):
return v.lower()

# TODO[pydantic]: The following keys were removed: `json_encoders`.
# Check https://docs.pydantic.dev/dev-v2/migration/#changes-to-config for more information.
# Check https://docs.pydantic.dev/dev-v2/migration/#changes-to-config for more information. # noqa: E501
model_config = ConfigDict(
arbitrary_types_allowed=True,
str_strip_whitespace=True,
Expand All @@ -96,7 +96,8 @@ def transform_uid(cls, v):
populate_by_name=True,
)

# Separate Coordinator & other members roles option in frontend, for better filtering for all_members_query
# Separate Coordinator & other members roles option in frontend,
# for better filtering for all_members_query


# TODO: ADD Descriptions for non-direct fields
11 changes: 7 additions & 4 deletions mutations.py
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,8 @@ def editMember(memberInput: FullMemberInput, info: Info) -> MemberType:
role["end_year"] = None
role_new = role.copy()

# if role's start_year, end_year, name is same as existing role, then keep the existing approved status
# if role's start_year, end_year, name is same as existing role,
# then keep the existing approved status
found_existing_role = False
for i in member_roles:
if (
Expand Down Expand Up @@ -166,7 +167,7 @@ def editMember(memberInput: FullMemberInput, info: Info) -> MemberType:
def deleteMember(memberInput: SimpleMemberInput, info: Info) -> MemberType:
"""
Mutation to delete an already existing member (role) of that specific 'club'
"""
""" # noqa: E501
user = info.context.user
if user is None:
raise Exception("Not Authenticated")
Expand Down Expand Up @@ -328,7 +329,8 @@ def rejectMember(memberInput: SimpleMemberInput, info: Info) -> MemberType:


# @strawberry.mutation
# def leaveClubMember(memberInput: SimpleMemberInput, info: Info) -> MemberType:
# def leaveClubMember(memberInput: SimpleMemberInput, info: Info) ->
# MemberType:
# user = info.context.user
# if user is None:
# raise Exception("Not Authenticated")
Expand All @@ -352,7 +354,8 @@ def rejectMember(memberInput: SimpleMemberInput, info: Info) -> MemberType:
# {"$set": {"end_year": datetime.now().year}},
# )

# created_sample = Member.parse_obj(membersdb.find_one({"_id": created_id}))
# created_sample = Member.parse_obj(membersdb.find_one(
# {"_id": created_id}))
# return MemberType.from_pydantic(created_sample)


Expand Down
2 changes: 1 addition & 1 deletion queries.py
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ def currentMembers(clubInput: SimpleClubInput, info: Info) -> List[MemberType]:
Scope: Anyone (Non-Admin Function)
Return Type: List[MemberType]
Input: SimpleClubInput (cid)
"""
""" # noqa: E501
user = info.context.user
if user is None:
role = "public"
Expand Down
3 changes: 2 additions & 1 deletion utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,8 @@
def non_deleted_members(member_input) -> MemberType:
"""
Function to return non-deleted members for a particular cid, uid
Only to be used in admin functions, as it returns both approved/non-approved members.
Only to be used in admin functions,
as it returns both approved/non-approved members.
"""
updated_sample = membersdb.find_one(
{
Expand Down

0 comments on commit c4fc3ad

Please sign in to comment.