I am writing some code to do simple checking of whether a person is a member (either directly or indirectly of a team). In my particular use case, I use a restricted team to keep track of people who have signed a contributor license agreement for the project, and I want to make lp:tarmac only merge branches from people who are a member of that team.
Currently I have this code snippet:
person = launchpad.people['statik']
contributor_team = launchpad.people['ubuntuone-contributor-agreement']
authorized_contributors = []
for p in contributor_team.participants:
authorized_contributors.append(p.name)
if person.name in authorized_contributors:
status = 'APPROVED CONTRIBUTOR'
else:
status = 'CONTRIBUTOR AGREEMENT NOT YET RECEIVED'
However, looking at the launchpad API documentation, I see a note on team.participants_collection_link referring to IPerson.inTeam. I believe this would be better if I could just do:
if person.inTeam(authorized_contributors):
Could you expose this method through the web API? Talking to sinzui today he thought it might be one line of code to add this.
I must abandon this. IPerson.inTeam is a privileged method and making work for external API is very difficult.
I recommend anyone working on this in the future to review the rejected work first:
https:/
I propose that instead of exposing inTeam, a new API-safe method be added (isInTeam()). The method could use the is_visible_by_user construct as a guard that calls the real inTeam() method.