Skip to content

Commit b55ec11

Browse files
connor peshekconnor peshek
authored andcommitted
Clean up integration.
1 parent 492101a commit b55ec11

File tree

2 files changed

+10
-18
lines changed

2 files changed

+10
-18
lines changed

web/src/admin/applications/wizard/ak-application-wizard-main.ts

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ import { WizardUpdateEvent } from "#components/ak-wizard/events";
1919
import { ProvidersApi, ProxyMode } from "@goauthentik/api";
2020

2121
import { ContextProvider } from "@lit/context";
22-
import { msg } from "@lit/localize";
2322
import { html } from "lit";
2423
import { customElement, state } from "lit/decorators.js";
2524

@@ -51,25 +50,18 @@ export class AkApplicationWizardMain extends AKElement {
5150
connectedCallback() {
5251
super.connectedCallback();
5352
new ProvidersApi(DEFAULT_CONFIG).providersAllTypesList().then((providerTypes) => {
54-
const wizardReadyProviders = Object.keys(providerTypeRenderers);
53+
// Filter out enterprise providers and those without wizard renderers
5554
const filteredProviders = providerTypes
56-
.filter((providerType) => wizardReadyProviders.includes(providerType.modelName))
57-
.map((providerType) => ({
58-
...providerType,
59-
renderer: providerTypeRenderers[providerType.modelName].render,
55+
.filter((type) => !type.requiresEnterprise)
56+
.filter((type) => {
57+
return type.modelName in providerTypeRenderers;
58+
})
59+
.map((type) => ({
60+
...type,
61+
renderer: providerTypeRenderers[type.modelName].render,
6062
}));
6163

62-
// Manually add SAML metadata provider since it's not a separate model type
63-
// but a different creation method for SAMLProvider
64-
filteredProviders.push({
65-
name: msg("SAML Provider from Metadata"),
66-
description: msg("Create a SAML provider by importing metadata from an XML file."),
67-
component: "ak-provider-saml-import-form",
68-
modelName: "samlmetadataprovider",
69-
iconUrl: "/static/authentik/sources/saml.png",
70-
renderer: providerTypeRenderers.samlmetadataprovider.render,
71-
});
72-
64+
// Sort by order and reverse to get highest order first
7365
this.wizardProviderProvider.setValue(
7466
filteredProviders
7567
.sort(

web/src/admin/applications/wizard/steps/ProviderChoices.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ export const providerTypeRenderers: Record<
3939
html`<ak-application-wizard-authentication-by-saml-configuration></ak-application-wizard-authentication-by-saml-configuration>`,
4040
order: 80,
4141
},
42-
samlmetadataprovider: {
42+
samlproviderimportmodel: {
4343
render: () =>
4444
html`<ak-application-wizard-authentication-by-saml-metadata-configuration></ak-application-wizard-authentication-by-saml-metadata-configuration>`,
4545
order: 100,

0 commit comments

Comments
 (0)