OB-2457 # Removed checkIsFormElementSupported property from ResourceDefinition type

CHANGELOG.md

@@ -7,9 +7,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
-### Changed
+### Removed
 
-- **[BREAKING]** input parameters to `ResourceDefinition['checkIsFormElementSupported']` function.
+- **[BREAKING]** `ResourceDefinition['checkIsFormElementSupported']` property
 
 ## [7.4.0] - 2025-03-11
 

src/freshdeskService.ts

@@ -106,15 +106,6 @@ export function generateFreshdeskTicketFieldDefinitions(
           type: 'CHOICE_SINGLE',
           ...baseField,
           choices: freshdeskField.options,
-          checkIsFormElementSupported(formElementWithOptions) {
-            if (freshdeskField.type === 'nested_field') {
-              return false
-            }
-
-            return (
-              formElementWithOptions.freshdeskFieldName === freshdeskField.name
-            )
-          },
         })
       } else if (
         freshdeskField.type === 'nested_field' &&

src/sharepointService.ts

@@ -44,27 +44,6 @@ export function generateSharepointColumnResourceDefinitions(
               label: choice,
               value: choice,
             })),
-            checkIsFormElementSupported(
-              formElementWithOptions,
-              { formsAppEnvironmentId, formElementOptionsSets },
-            ) {
-              const formElementOptionsSetId =
-                formElementWithOptions.dynamicOptionSetId
-              return (
-                !!formElementOptionsSetId &&
-                formElementWithOptions.optionsType === 'DYNAMIC' &&
-                formElementOptionsSets.some((formElementOptionsSet) => {
-                  return (
-                    formElementOptionsSet.type === 'SHAREPOINT_LIST_COLUMN' &&
-                    formElementOptionsSet.environments.some(
-                      (env) =>
-                        env.formsAppEnvironmentId === formsAppEnvironmentId &&
-                        env.sharepointColumn.id === columnDefinition.id,
-                    )
-                  )
-                })
-              )
-            },
           })
         } else if (columnDefinition.dateTime?.format === 'dateOnly') {
           memo.push({

src/types/resource-definitions.ts

@@ -1,15 +1,22 @@
-import { FormTypes } from '@oneblink/types'
-
 export type ResourceDefinitionChoice = {
   value: string | number
   label: string
 }
 
-export type ResourceDefinition<T> = T & {
+type ResourceDefinitionBase = {
   id: string
   isRequired: boolean
   displayName: string
-} & (
+}
+
+export interface ResourceDefinitionWithChoices {
+  type: 'CHOICE_SINGLE' | 'CHOICE_MULTIPLE'
+  choices: ResourceDefinitionChoice[]
+}
+
+export type ResourceDefinition<T> = T &
+  ResourceDefinitionBase &
+  (
     | {
         type:
           | 'TEXT_SINGLE_LINE'
@@ -20,15 +27,5 @@ export type ResourceDefinition<T> = T & {
           | 'NUMBER'
           | 'BOOLEAN'
       }
-    | {
-        type: 'CHOICE_SINGLE' | 'CHOICE_MULTIPLE'
-        choices: ResourceDefinitionChoice[]
-        checkIsFormElementSupported: (
-          formElementWithOptions: FormTypes.FormElementWithOptions,
-          options: {
-            formElementOptionsSets: FormTypes.FormElementOptionSet[]
-            formsAppEnvironmentId: number
-          },
-        ) => boolean
-      }
+    | ResourceDefinitionWithChoices
   )

tests/freshdesk.test.ts

@@ -71,7 +71,6 @@ describe('generateFreshdeskTicketFieldDefinitions', () => {
     ]
     expect(generateFreshdeskTicketFieldDefinitions(fields)).toEqual([
       {
-        checkIsFormElementSupported: expect.any(Function),
         choices: [
           {
             label: 'option 1',
@@ -259,7 +258,6 @@ describe('generateFreshdeskTicketFieldDefinitions', () => {
     ]
     expect(generateFreshdeskTicketFieldDefinitions(fields)).toEqual([
       {
-        checkIsFormElementSupported: expect.any(Function),
         choices: [
           {
             label: 'value 1',

tests/sharepointService.test.ts

@@ -80,7 +80,6 @@ describe('generateSharepointColumnResourceDefinitions', () => {
           { label: 'A', value: 'A' },
           { label: 'B', value: 'B' },
         ],
-        checkIsFormElementSupported: expect.any(Function),
       },
     ])
   })