diff --git a/passbook/core/templates/user/base.html b/passbook/core/templates/user/base.html
index 4c0c6798b..944684480 100644
--- a/passbook/core/templates/user/base.html
+++ b/passbook/core/templates/user/base.html
@@ -25,8 +25,7 @@
{% for stage in user_stages_loc %}
-
-
-
+
{{ stage.name }}
@@ -43,7 +42,6 @@
-
-
{{ source.name }}
diff --git a/passbook/core/types.py b/passbook/core/types.py
index 86d6eeea6..7804fcb29 100644
--- a/passbook/core/types.py
+++ b/passbook/core/types.py
@@ -8,8 +8,7 @@ class UIUserSettings:
"""Dataclass for Stage and Source's user_settings"""
name: str
- icon: str
- view_name: str
+ url: str
@dataclass
diff --git a/passbook/flows/views.py b/passbook/flows/views.py
index 0182afa53..fa670b8be 100644
--- a/passbook/flows/views.py
+++ b/passbook/flows/views.py
@@ -15,6 +15,7 @@ from django.views.decorators.clickjacking import xframe_options_sameorigin
from django.views.generic import TemplateView, View
from structlog import get_logger
+from passbook.audit.models import sanitize_dict
from passbook.core.views.utils import PermissionDeniedView
from passbook.flows.exceptions import EmptyFlowException, FlowNonApplicableException
from passbook.flows.models import Flow, FlowDesignation, Stage
@@ -161,7 +162,7 @@ class FlowExecutorView(View):
LOGGER.debug(
"f(exec): User passed all stages",
flow_slug=self.flow.slug,
- context=self.plan.context,
+ context=sanitize_dict(self.plan.context),
)
return self._flow_done()
diff --git a/passbook/sources/oauth/models.py b/passbook/sources/oauth/models.py
index 43f2a6280..09058c041 100644
--- a/passbook/sources/oauth/models.py
+++ b/passbook/sources/oauth/models.py
@@ -62,15 +62,9 @@ class OAuthSource(Source):
@property
def ui_user_settings(self) -> UIUserSettings:
- icon_type = self.provider_type
- if icon_type == "azure ad":
- icon_type = "windows"
- icon_class = f"fab fa-{icon_type}"
view_name = "passbook_sources_oauth:oauth-client-user"
return UIUserSettings(
- name=self.name,
- icon=icon_class,
- view_name=reverse((view_name), kwargs={"source_slug": self.slug}),
+ name=self.name, url=reverse(view_name, kwargs={"source_slug": self.slug}),
)
def __str__(self) -> str:
diff --git a/passbook/stages/otp/models.py b/passbook/stages/otp/models.py
index f0a6e1c88..10015be61 100644
--- a/passbook/stages/otp/models.py
+++ b/passbook/stages/otp/models.py
@@ -1,5 +1,6 @@
"""OTP Stage"""
from django.db import models
+from django.urls import reverse
from django.utils.translation import gettext as _
from passbook.core.types import UIUserSettings
@@ -20,9 +21,7 @@ class OTPStage(Stage):
@property
def ui_user_settings(self) -> UIUserSettings:
return UIUserSettings(
- name="OTP",
- icon="pficon-locked",
- view_name="passbook_stages_otp:otp-user-settings",
+ name="OTP", url=reverse("passbook_stages_otp:otp-user-settings"),
)
def __str__(self):