diff --git a/apps/console/src/components/obligations/LinkedObligationsCard.tsx b/apps/console/src/components/obligations/LinkedObligationsCard.tsx index 575c1479e..2c7037613 100644 --- a/apps/console/src/components/obligations/LinkedObligationsCard.tsx +++ b/apps/console/src/components/obligations/LinkedObligationsCard.tsx @@ -36,7 +36,6 @@ import { import { clsx } from "clsx"; import { useMemo, useState } from "react"; import { useFragment } from "react-relay"; -import { useParams } from "react-router"; import { graphql } from "relay-runtime"; import type { LinkedObligationsCardFragment$key } from "#/__generated__/core/LinkedObligationsCardFragment.graphql"; @@ -203,16 +202,12 @@ function ObligationRow(props: { const { __ } = useTranslate(); const obligation = useFragment(linkedObligationFragment, props.obligation); const organizationId = useOrganizationId(); - const { snapshotId } = useParams<{ snapshotId?: string }>(); - const isSnapshotMode = Boolean(snapshotId); const onDetach = () => { props.onClick(obligation.id); }; - const detailsUrl = isSnapshotMode - ? `/organizations/${organizationId}/snapshots/${snapshotId}/obligations/${obligation.id}` - : `/organizations/${organizationId}/obligations/${obligation.id}`; + const detailsUrl = `/organizations/${organizationId}/obligations/${obligation.id}`; return ( diff --git a/apps/console/src/hooks/graph/ObligationGraph.ts b/apps/console/src/hooks/graph/ObligationGraph.ts index 975956df0..ebdc973d8 100644 --- a/apps/console/src/hooks/graph/ObligationGraph.ts +++ b/apps/console/src/hooks/graph/ObligationGraph.ts @@ -25,11 +25,18 @@ import { useMutationWithToasts } from "../useMutationWithToasts"; export const ObligationsConnectionKey = "ObligationsPage_obligations"; export const obligationsQuery = graphql` - query ObligationGraphListQuery($organizationId: ID!, $snapshotId: ID) { + query ObligationGraphListQuery($organizationId: ID!) { node(id: $organizationId) { ... on Organization { canCreateObligation: permission(action: "core:obligation:create") - ...ObligationsPageFragment @arguments(snapshotId: $snapshotId) + canPublishObligations: permission(action: "core:obligation:publish") + obligationsDocument { + id + defaultApprovers { + id + } + } + ...ObligationsPageFragment } } } @@ -40,8 +47,6 @@ export const obligationNodeQuery = graphql` node(id: $obligationId) { ... on Obligation { id - snapshotId - sourceId area source requirement diff --git a/apps/console/src/pages/organizations/findings/FindingDetailsPage.tsx b/apps/console/src/pages/organizations/findings/FindingDetailsPage.tsx index be062e2e2..5787d9d85 100644 --- a/apps/console/src/pages/organizations/findings/FindingDetailsPage.tsx +++ b/apps/console/src/pages/organizations/findings/FindingDetailsPage.tsx @@ -20,7 +20,6 @@ import { getStatusVariant, type GraphQLError, sprintf, - validateSnapshotConsistency, } from "@probo/helpers"; import { useTranslate } from "@probo/i18n"; import { @@ -48,7 +47,6 @@ import { useMutation, usePreloadedQuery, } from "react-relay"; -import { useParams } from "react-router"; import { z } from "zod"; import type { FindingDetailsPageDeleteMutation } from "#/__generated__/core/FindingDetailsPageDeleteMutation.graphql"; @@ -56,7 +54,6 @@ import type { FindingDetailsPageQuery } from "#/__generated__/core/FindingDetail import type { FindingDetailsPageUpdateMutation } from "#/__generated__/core/FindingDetailsPageUpdateMutation.graphql"; import { ControlledField } from "#/components/form/ControlledField"; import { PeopleSelectField } from "#/components/form/PeopleSelectField"; -import { SnapshotBanner } from "#/components/SnapshotBanner"; import { useFormWithSchema } from "#/hooks/useFormWithSchema"; import { useOrganizationId } from "#/hooks/useOrganizationId"; @@ -67,7 +64,6 @@ export const findingDetailsPageQuery = graphql` node(id: $findingId) { ... on Finding { id - snapshotId kind referenceId description @@ -166,12 +162,8 @@ export default function FindingDetailsPage(props: Props) { const { __ } = useTranslate(); const { toast } = useToast(); const organizationId = useOrganizationId(); - const { snapshotId } = useParams<{ snapshotId?: string }>(); - const isSnapshotMode = Boolean(snapshotId); const confirm = useConfirm(); - validateSnapshotConsistency(finding, snapshotId); - const [updateFinding] = useMutation(updateFindingMutation); const [deleteFinding] = useMutation(deleteFindingMutation); @@ -181,7 +173,6 @@ export default function FindingDetailsPage(props: Props) { FindingsConnectionKey, { filter: { - snapshotId: snapshotId || null, kind: null, status: null, priority: null, @@ -194,7 +185,6 @@ export default function FindingDetailsPage(props: Props) { FindingsConnectionKey, { filter: { - snapshotId: snapshotId || null, kind: finding.kind, status: null, priority: null, @@ -322,15 +312,10 @@ export default function FindingDetailsPage(props: Props) { { value: "HIGH", label: __("High") }, ]; - const breadcrumbFindingsUrl = isSnapshotMode - ? `/organizations/${organizationId}/snapshots/${snapshotId}/findings` - : `/organizations/${organizationId}/findings`; + const breadcrumbFindingsUrl = `/organizations/${organizationId}/findings`; return (
- {isSnapshotMode && snapshotId && ( - - )}
- {!isSnapshotMode && ( - - {finding.canDelete && ( - - {__("Delete")} - - )} - - )} + + {finding.canDelete && ( + + {__("Delete")} + + )} +
@@ -393,7 +376,6 @@ export default function FindingDetailsPage(props: Props) { {...register("description")} placeholder={__("Enter description")} rows={3} - disabled={isSnapshotMode} /> @@ -405,7 +387,6 @@ export default function FindingDetailsPage(props: Props) { @@ -416,7 +397,6 @@ export default function FindingDetailsPage(props: Props) { label={__("Owner")} error={formState.errors.ownerId?.message} optional - disabled={isSnapshotMode} />
@@ -427,7 +407,6 @@ export default function FindingDetailsPage(props: Props) { type="select" label={__("Status")} required - disabled={isSnapshotMode} > {statusOptions.map(option => (