improvements: fix doc endpoint and minor ui adjustments/permission check

This commit is contained in:
Scott Wilson
2025-11-11 23:34:48 -05:00
committed by =
parent 45b89f592e
commit a9765e125b
4 changed files with 61 additions and 44 deletions

View File

@@ -1,4 +1,4 @@
---
title: "Get Identity By Id"
title: "Get Identity by ID"
openapi: "GET /api/v1/projects/{projectId}/identities/{identityId}"
---

View File

@@ -1,4 +1,4 @@
---
title: "List Identity"
openapi: "POST /api/v1/projects/{projectId}/identities"
title: "List Identities"
openapi: "GET /api/v1/projects/{projectId}/identities"
---

View File

@@ -9,7 +9,7 @@ import { cva, type VariantProps } from "cva";
import { twMerge } from "tailwind-merge";
const alertVariants = cva(
"w-full bg-mineshaft-800 rounded-lg border border-bunker-400 px-4 py-3 text-sm flex items-center gap-x-4",
"w-full bg-mineshaft-800 rounded-lg border border-bunker-400 px-4 py-3 text-sm flex items-center gap-x-3",
{
variants: {
variant: {
@@ -28,6 +28,7 @@ type AlertProps = {
title?: string;
hideTitle?: boolean;
icon?: React.ReactNode;
iconClassName?: string;
};
const variantTitleMap = {
@@ -45,36 +46,41 @@ const variantIconMap = {
const Alert = forwardRef<
HTMLDivElement,
React.HTMLAttributes<HTMLDivElement> & VariantProps<typeof alertVariants> & AlertProps
>(({ className, variant, title, icon, hideTitle = false, children, ...props }, ref) => {
const defaultTitle = title ?? variantTitleMap[variant ?? "default"];
return (
<div
ref={ref}
role="alert"
className={twMerge(alertVariants({ variant }), className)}
{...props}
>
<div>
{typeof icon !== "undefined" ? (
<>{icon} </>
) : (
<FontAwesomeIcon
className="text-lg text-primary"
icon={variantIconMap[variant ?? "default"]}
/>
)}
>(
(
{ className, variant, title, icon, hideTitle = false, children, iconClassName, ...props },
ref
) => {
const defaultTitle = title ?? variantTitleMap[variant ?? "default"];
return (
<div
ref={ref}
role="alert"
className={twMerge(alertVariants({ variant }), className)}
{...props}
>
<div>
{typeof icon !== "undefined" ? (
<>{icon} </>
) : (
<FontAwesomeIcon
className={twMerge("text-lg text-primary", iconClassName)}
icon={variantIconMap[variant ?? "default"]}
/>
)}
</div>
<div className="flex flex-col gap-y-1">
{hideTitle ? null : (
<h5 className="leading-6 font-medium tracking-tight" {...props}>
{defaultTitle}
</h5>
)}
{children}
</div>
</div>
<div className="flex flex-col gap-y-1">
{hideTitle ? null : (
<h5 className="leading-6 font-medium tracking-tight" {...props}>
{defaultTitle}
</h5>
)}
{children}
</div>
</div>
);
});
);
}
);
Alert.displayName = "Alert";
const AlertDescription = forwardRef<

View File

@@ -8,7 +8,7 @@ import { Link, useNavigate, useParams } from "@tanstack/react-router";
import { formatRelative } from "date-fns";
import { createNotification } from "@app/components/notifications";
import { ProjectPermissionCan } from "@app/components/permissions";
import { OrgPermissionCan, ProjectPermissionCan } from "@app/components/permissions";
import {
Alert,
AlertDescription,
@@ -20,6 +20,8 @@ import {
Spinner
} from "@app/components/v2";
import {
OrgPermissionIdentityActions,
OrgPermissionSubjects,
ProjectPermissionActions,
ProjectPermissionIdentityActions,
ProjectPermissionSub,
@@ -211,19 +213,28 @@ const Page = () => {
</div>
</PageHeader>
{!isProjectIdentity && (
<Alert hideTitle className="mb-4 border-primary/50 bg-primary/10">
<Alert hideTitle iconClassName="text-info" className="mb-4 border-info/50 bg-info/10">
<AlertDescription>
This identity is managed by your organization.{" "}
<Link
to="/organization/identities/$identityId"
params={{
identityId
}}
<OrgPermissionCan
I={OrgPermissionIdentityActions.Read}
an={OrgPermissionSubjects.Identity}
>
<span className="cursor-pointer text-primary underline underline-offset-2">
Click here to retrieve credentials.
</span>
</Link>
{(isAllowed) =>
isAllowed ? (
<Link
to="/organization/identities/$identityId"
params={{
identityId
}}
>
<span className="cursor-pointer text-info underline underline-offset-2">
Click here to manage identity.
</span>
</Link>
) : null
}
</OrgPermissionCan>
</AlertDescription>
</Alert>
)}