libre.sh/keycloak-scim
4
0
Fork 0
Code Issues 15 Pull requests Releases 1 Activity Actions

Redirect to flow detail after duplicate (#3199)

Browse source
This commit is contained in:
kris2kris 2022-08-29 14:40:36 +02:00 committed by GitHub
parent fd29c77928
commit 5ad85d38da
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
apps/admin-ui/cypress/e2e/authentication_test.spec.ts
View file

@ -52,7 +52,7 @@ describe("Authentication test", () => {
duplicateFlowModal.fill("Copy of browser"); duplicateFlowModal.fill("Copy of browser");
masthead.checkNotificationMessage("Flow successfully duplicated"); masthead.checkNotificationMessage("Flow successfully duplicated");
listingPage.itemExist("Copy of browser"); detailPage.flowExists("Copy of browser");
}); });
it("Should fail duplicate with empty flow name", () => { it("Should fail duplicate with empty flow name", () => {

15
apps/admin-ui/src/authentication/DuplicateFlowModal.tsx
View file

@ -1,6 +1,7 @@
import { useEffect } from "react"; import { useEffect } from "react";
import { useTranslation } from "react-i18next"; import { useTranslation } from "react-i18next";
import { FormProvider, useForm } from "react-hook-form"; import { FormProvider, useForm } from "react-hook-form";
import { useNavigate } from "react-router-dom-v5-compat";
import { import {
AlertVariant, AlertVariant,
Button, Button,
@ -13,6 +14,8 @@ import {
import { useAdminClient } from "../context/auth/AdminClient"; import { useAdminClient } from "../context/auth/AdminClient";
import { useAlerts } from "../components/alert/Alerts"; import { useAlerts } from "../components/alert/Alerts";
import { NameDescription } from "./form/NameDescription"; import { NameDescription } from "./form/NameDescription";
import { toFlow } from "./routes/Flow";
import { useRealm } from "../context/realm-context/RealmContext";
type DuplicateFlowModalProps = { type DuplicateFlowModalProps = {
name: string; name: string;
@ -34,6 +37,8 @@ export const DuplicateFlowModal = ({
const { setValue, trigger, getValues } = form; const { setValue, trigger, getValues } = form;
const { adminClient } = useAdminClient(); const { adminClient } = useAdminClient();
const { addAlert, addError } = useAlerts(); const { addAlert, addError } = useAlerts();
const navigate = useNavigate();
const { realm } = useRealm();
useEffect(() => { useEffect(() => {
setValue("description", description); setValue("description", description);
@ -48,11 +53,11 @@ export const DuplicateFlowModal = ({
flow: name, flow: name,
newName: form.alias, newName: form.alias,
}); });
if (form.description !== description) {
const newFlow = ( const newFlow = (
await adminClient.authenticationManagement.getFlows() await adminClient.authenticationManagement.getFlows()
).find((flow) => flow.alias === form.alias)!; ).find((flow) => flow.alias === form.alias)!;
if (form.description !== description) {
newFlow.description = form.description; newFlow.description = form.description;
await adminClient.authenticationManagement.updateFlow( await adminClient.authenticationManagement.updateFlow(
{ flowId: newFlow.id! }, { flowId: newFlow.id! },
@ -60,6 +65,14 @@ export const DuplicateFlowModal = ({
); );
} }
addAlert(t("copyFlowSuccess"), AlertVariant.success); addAlert(t("copyFlowSuccess"), AlertVariant.success);
navigate(
toFlow({
realm,
id: newFlow.id!,
usedBy: "notInUse",
builtIn: newFlow.builtIn ? "builtIn" : undefined,
})
);
} catch (error) { } catch (error) {
addError("authentication:copyFlowError", error); addError("authentication:copyFlowError", error);
} }