[SYNPY-1633] Add membership_status to Team model and Deprecate Old Method

[linglp]

Problem:

This method here needs to be deprecated: synapsePythonClient/synapseclient/client.py at 02d3b38d2a0f043811637064f61ae8b537e4253b · Sage-Bionetworks/synapsePythonClient

Solution:

Added a new method called get_user_membership_status under team

Testing:

• Ensured that the example in the docstring works
• Surfaced the new function in documentation
• Added test

[BryanFauble]

Since users will call this, can we convert the response from the API call (https://rest-docs.synapse.org/rest/org/sagebionetworks/repo/model/TeamMembershipStatus.html) into a Dataclass (Similar to what we did with TeamMembers

synapsePythonClient/synapseclient/models/team.py

Lines 23 to 56 in 04cfbe3

	class TeamMember:
	"""
	Contains information about a user's membership in a Team.
	In practice the constructor is not called directly by the client.
	Attributes:
	team_id: The ID of the team
	member: An object of type [org.sagebionetworks.repo.model.UserGroupHeader](https://rest-docs.synapse.org/rest/org/sagebionetworks/repo/model/UserGroupHeader.html)
	describing the member
	is_admin: Whether the given member is an administrator of the team
	"""
	team_id: Optional[int] = None
	"""The ID of the team"""
	member: Optional[UserGroupHeader] = None
	"""An object of type [org.sagebionetworks.repo.model.UserGroupHeader](https://rest-docs.synapse.org/rest/org/sagebionetworks/repo/model/UserGroupHeader.html)"""
	is_admin: Optional[bool] = None
	"""Whether the given member is an administrator of the team"""
	def fill_from_dict(
	self, synapse_team_member: Union[Synapse_TeamMember, Dict[str, str]]
	) -> "TeamMember":
	self.team_id = (
	int(synapse_team_member.get("teamId", None))
	if synapse_team_member.get("teamId", None)
	else None
	)
	self.member = UserGroupHeader().fill_from_dict(
	synapse_team_member.get("member", None)
	)
	self.is_admin = synapse_team_member.get("isAdmin", None)
	return self

)

This will ensure that anyone calling this Method knows exactly what to expect from both the mkdocs page AND within the IDE via the hoverdocs.

[BryanFauble]

Swap to Dataclass for return from method for better user experience

[BryanFauble]

This is great! I had a few comments, but nothing that changes the functionality of what was proposed.