From 5526356938eb3f33d2d6e9c9c34f677190212202 Mon Sep 17 00:00:00 2001 From: graham-dds Date: Mon, 28 Oct 2019 14:01:54 -0400 Subject: [PATCH] remove TODO from admin.py --- atst/routes/portfolios/admin.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/atst/routes/portfolios/admin.py b/atst/routes/portfolios/admin.py index a17d23b2..187ca714 100644 --- a/atst/routes/portfolios/admin.py +++ b/atst/routes/portfolios/admin.py @@ -185,14 +185,13 @@ def remove_member(portfolio_id, portfolio_role_id): raise UnauthorizedError( g.current_user, "you can't delete the portfolios PPoC from the portfolio" ) + if ( portfolio_role.latest_invitation and portfolio_role.status == PortfolioRoleStatus.PENDING ): PortfolioInvitations.revoke(portfolio_role.latest_invitation.token) else: - # TODO: should this cascade and disable any application and environment - # roles they might have? PortfolioRoles.disable(portfolio_role=portfolio_role) flash("portfolio_member_removed", member_name=portfolio_role.full_name)