made vikunja team sync only happen if user active

membermatters · May 30, 2024 · bcc191e · bcc191e
1 parent dcc17b3
commit bcc191e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/memberportal/membermatters/constance_config.py b/memberportal/membermatters/constance_config.py
@@ -419,7 +419,7 @@
                 "MEMBERBUCKS_CURRENCY",
             ),
         ),
-        ("Vikunja Integration", ("VIKUNJA_TEAMS")),
+        ("Vikunja Integration", ("VIKUNJA_TEAMS",)),
         (
             "Trello Integration",
             (

diff --git a/memberportal/membermatters/oidc_provider_settings.py b/memberportal/membermatters/oidc_provider_settings.py
@@ -69,10 +69,14 @@ def scope_vikunja_teams(self):
         if config.VIKUNJA_TEAMS:
             try:
                 teams = json.loads(config.VIKUNJA_TEAMS)
-
-                return {
-                    "vikunja_groups": teams,
-                }
+                if self.user.profile.state == "active":
+                    return {
+                        "vikunja_groups": teams,
+                    }
+                else:
+                    return {
+                        "vikunja_groups": [],
+                    }
             except json.JSONDecodeError:
                 logger.error(
                     "VIKUNJA_TEAMS is not a valid JSON object and the Vikunja teams claim wasn't added."