feat: fixed test, resolved another edgecase in dashboard and added label to conditions in secrets

backend/e2e-test/routes/v3/secrets.spec.ts

@@ -812,6 +812,8 @@ describe.each([{ auth: AuthMode.JWT }, { auth: AuthMode.IDENTITY_ACCESS_TOKEN }]
         },
         body: createSecretReqBody
       });
+      console.log(">>>>>>>>>>");
+      console.log(createSecRes.payload);
       expect(createSecRes.statusCode).toBe(200);
       const createdSecretPayload = JSON.parse(createSecRes.payload);
       expect(createdSecretPayload).toHaveProperty("secret");

backend/src/ee/services/permission/permission-dal.ts

@@ -441,7 +441,7 @@ export const permissionDALFactory = (db: TDbClient) => {
             .on(`${TableName.Identity}.id`, `${TableName.IdentityMetadata}.identityId`)
             .andOn(`${TableName.Project}.orgId`, `${TableName.IdentityMetadata}.orgId`);
         })
-        .where("identityId", identityId)
+        .where(`${TableName.IdentityProjectMembership}.identityId`, identityId)
         .where(`${TableName.IdentityProjectMembership}.projectId`, projectId)
         .select(selectAllTableCols(TableName.IdentityProjectMembershipRole))
         .select(

backend/src/server/routes/v1/dashboard-router.ts

@@ -200,7 +200,7 @@ export const registerDashboardRouter = async (server: FastifyZodProvider) => {
           )
         );
 
-      if (includeDynamicSecrets) {
+      if (includeDynamicSecrets && permissiveEnvs.length) {
         // this is the unique count, ie duplicate secrets across envs only count as 1
         totalDynamicSecretCount = await server.services.dynamicSecret.getCountMultiEnv({
           actor: req.permission.type,
@@ -241,7 +241,7 @@ export const registerDashboardRouter = async (server: FastifyZodProvider) => {
         }
       }
 
-      if (includeSecrets) {
+      if (includeSecrets && permissiveEnvs.length) {
         // this is the unique count, ie duplicate secrets across envs only count as 1
         totalSecretCount = await server.services.secret.getSecretsCountMultiEnv({
           actorId: req.permission.id,

frontend/src/views/Project/RolePage/components/RolePermissionsSection/components/GeneralPermissionOptions.tsx

@@ -93,6 +93,7 @@ export const GeneralPermissionOptions = <T extends keyof NonNullable<TFormSchema
                             render={({ field }) => (
                               <div className="flex items-center justify-center">
                                 <Checkbox
+                                  isDisabled={isDisabled}
                                   isChecked={Boolean(field.value)}
                                   onCheckedChange={field.onChange}
                                   id={`permissions.${subject}.${rootIndex}.${String(value)}`}

frontend/src/views/Project/RolePage/components/RolePermissionsSection/components/SecretPermissionConditions.tsx

@@ -12,8 +12,14 @@ type Props = {
   isDisabled?: boolean;
 };
 
+const getValueLabel = (type: string) => {
+  if (type === "environment") return "Environment slug";
+  if (type === "secretPath") return "Folder path";
+  return "";
+};
+
 export const SecretPermissionConditions = ({ position = 0, isDisabled }: Props) => {
-  const { control } = useFormContext<TFormSchema>();
+  const { control, watch } = useFormContext<TFormSchema>();
   const items = useFieldArray({
     control,
     name: `permissions.secrets.${position}.conditions`
@@ -22,7 +28,9 @@ export const SecretPermissionConditions = ({ position = 0, isDisabled }: Props)
   return (
     <div className="mt-6 border-t border-t-gray-800 bg-mineshaft-800 pt-2">
       <div className="mt-2 flex flex-col space-y-2">
-        {items.fields.map((el, index) => (
+        {items.fields.map((el, index) => {
+          const lhs = watch(`permissions.secrets.${position}.conditions.${index}.lhs`);
+          return (
             <div
               key={el.id}
               className="flex gap-2 bg-mineshaft-800 first:rounded-t-md last:rounded-b-md"
@@ -68,7 +76,9 @@ export const SecretPermissionConditions = ({ position = 0, isDisabled }: Props)
                       >
                         <SelectItem value={PermissionConditionOperators.$EQ}>Equal</SelectItem>
                         <SelectItem value={PermissionConditionOperators.$NEQ}>Not Equal</SelectItem>
-                      <SelectItem value={PermissionConditionOperators.$GLOB}>Glob Match</SelectItem>
+                        <SelectItem value={PermissionConditionOperators.$GLOB}>
+                          Glob Match
+                        </SelectItem>
                         <SelectItem value={PermissionConditionOperators.$IN}>Contains</SelectItem>
                       </Select>
                     </FormControl>
@@ -85,7 +95,7 @@ export const SecretPermissionConditions = ({ position = 0, isDisabled }: Props)
                       errorText={error?.message}
                       className="mb-0 flex-grow"
                     >
-                    <Input {...field} placeholder="value" />
+                      <Input {...field} placeholder={getValueLabel(lhs)} />
                     </FormControl>
                   )}
                 />
@@ -101,7 +111,8 @@ export const SecretPermissionConditions = ({ position = 0, isDisabled }: Props)
                 </IconButton>
               </div>
             </div>
-        ))}
+          );
+        })}
       </div>
       <div>
         <Button

frontend/src/views/SecretOverviewPage/SecretOverviewPage.tsx

@@ -190,6 +190,7 @@ export const SecretOverviewPage = () => {
   }, [isWorkspaceLoading, workspaceId, router.isReady]);
 
   const userAvailableEnvs = currentWorkspace?.environments || [];
+
   const [visibleEnvs, setVisibleEnvs] = useState(userAvailableEnvs);
 
   useEffect(() => {