Skip to content

Commit 215de12

Browse files
authored
Merge pull request #2492 from JohnDuprey/dev
Fix scripted alerts
2 parents ed01c11 + 2e67192 commit 215de12

File tree

4 files changed

+10
-6
lines changed

4 files changed

+10
-6
lines changed

src/components/forms/RFFComponents.jsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -587,7 +587,11 @@ export const RFFSelectSearch = ({
587587
{...props}
588588
/>
589589
)}
590-
{meta.error && meta.touched && <span className="text-danger">{meta.error}</span>}
590+
{meta.error && meta.touched && (
591+
<span className="text-danger">
592+
{typeof meta.error === 'object' ? Object.values(meta.error).join('') : meta.error}
593+
</span>
594+
)}
591595
</div>
592596
)
593597
}}

src/views/cipp/app-settings/SettingsSuperAdmin.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ export function SettingsSuperAdmin() {
103103
</>
104104
</>
105105
</CippAccordionItem>
106-
<CippAccordionItem title="Custom Roles" titleType="big">
106+
<CippAccordionItem title="Custom Roles (beta)" titleType="big">
107107
<SettingsCustomRoles />
108108
</CippAccordionItem>
109109
</CAccordion>

src/views/cipp/app-settings/components/SettingsCustomRoles.jsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -227,8 +227,9 @@ const SettingsCustomRoles = () => {
227227
direct API access, create a role with the name 'CIPP-API'.
228228
</p>
229229
<p className="small">
230-
NOTE: The custom role must be added to the user in SWA in conjunction with the base role.
231-
(e.g. editor,mycustomrole)
230+
<FontAwesomeIcon icon="triangle-exclamation" className="me-2" /> This functionality is in
231+
beta and should be treated as such. The custom role must be added to the user in SWA in
232+
conjunction with the base role. (e.g. editor,mycustomrole)
232233
</p>
233234
{(isFetching || tenantsFetching) && <Skeleton count={1} />}
234235
{isSuccess && !isFetching && !tenantsFetching && (

src/views/tenant/administration/AlertWizard.jsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,6 @@ const AlertWizard = () => {
9696
const getRecurrenceOptions = () => {
9797
const values = currentFormState?.values
9898
if (values) {
99-
//console.log(currentFormState)
10099
const updatedRecurrenceOptions = recurrenceOptions.map((opt) => ({
101100
...opt,
102101
name: opt.name.replace(' (Recommended)', ''),
@@ -317,7 +316,7 @@ const AlertWizard = () => {
317316
multi={true}
318317
name={`actions`}
319318
placeholder={
320-
'Select one action or multple actions from the list'
319+
'Select one action or multiple actions from the list'
321320
}
322321
label="Then perform the following action(s)"
323322
/>

0 commit comments

Comments
 (0)