Fixed status for enabled registrations

TIER · Dec 1, 2022 · 3aebda8 · 3aebda8
1 parent 893f006
commit 3aebda8
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/ui/src/app/dynamic-registration/component/DynamicRegistrationList.js b/ui/src/app/dynamic-registration/component/DynamicRegistrationList.js
@@ -77,7 +77,7 @@ export function DynamicRegistrationList ({entities, children, onChangeGroup, onD
                                         </td>
                                         <td className="text-center align-middle">
                                             <span className="d-flex justify-content-center align-items-center">
-                                            {onEnable && canEnable(reg.approved) ?
+                                            {!reg.enabled &&onEnable && canEnable(reg.approved) ?
                                                 <Form.Check
                                                     type="switch"
                                                     id={`enable-switch-${idx}`}
@@ -117,7 +117,7 @@ export function DynamicRegistrationList ({entities, children, onChangeGroup, onD
                                             <OverlayTrigger trigger={reg.enabled ? ['hover', 'focus'] : []} placement="left"
                                                 overlay={
                                                     <Popover id={`delete-source-btn-${idx}`}>
-                                                        <Popover.Body>A metadata source must be disabled before it can be deleted.</Popover.Body>
+                                                        <Popover.Body>An enabled registration cannot be deleted.</Popover.Body>
                                                     </Popover>
                                                 }>
                                                 <span className="d-inline-block">

diff --git a/ui/src/app/dynamic-registration/view/DynamicRegistrationDetail.js b/ui/src/app/dynamic-registration/view/DynamicRegistrationDetail.js
@@ -98,6 +98,7 @@ export function DynamicRegistrationDetail () {
                                                     </GroupsProvider>
                                                 }
                                             </h5>
+                                            {!detail.enabled &&
                                             <div className="d-flex align-items-start btn-group">
                                                 {enable && (canEnable && detail?.approved) &&
                                                     <Button variant={detail.enabled ? 'outline-secondary' : 'outline-secondary' } size="sm" className=""
@@ -119,6 +120,7 @@ export function DynamicRegistrationDetail () {
                                                     </Button>
                                                 }
                                             </div>
+                                            }
                                         </div>
 
                                         <p className="card-text">
@@ -134,7 +136,7 @@ export function DynamicRegistrationDetail () {
                                         {(schema) => 
                                         <Configuration entities={[detail]} schema={schema} definition={definition}>
                                             {(config) =>
-                                                <MetadataConfiguration configuration={config} onEdit={() => edit()} />
+                                                <MetadataConfiguration configuration={config} onEdit={!detail.enabled ? () => edit() : null} />
                                             }
                                         </Configuration>
                                     }

diff --git a/ui/src/app/store/dynamic-registration/DynamicRegistrationSlice.js b/ui/src/app/store/dynamic-registration/DynamicRegistrationSlice.js
@@ -12,13 +12,13 @@ export const DynamicRegistrationApi = createApi({
       query: () => ({
         url: `/DynamicRegistrations`
       }),
-      providesTags: ['DynamicRegistration']
+      providesTags: ['DynamicRegistration'],
     }),
     selectDynamicRegistration: builder.query({
       query: ({id}) => ({
         url: `/DynamicRegistration/${id}`
       }),
-      providesTags: ['DynamicRegistration']
+      providesTags: ['DynamicRegistration'],
     }),
     getDisabledRegistrations: builder.query({
       query: () => ({