Remove number from label identifier list

[prastoin]

Introduction

closes #12780
Removed the number field metadata from the selectable field metadata as identifier

Implemented server side validation and basic coverage on that

[greptile-apps]

PR Summary

Removes FieldMetadataType.NUMBER from label identifier types, restricting record identifiers to only text and full name fields. This change aligns the data model with its usage as record text in the UI.

• Removed NUMBER from LABEL_IDENTIFIER_FIELD_METADATA_TYPES in src/modules/object-metadata/constants/LabelIdentifierFieldMetadataTypes.ts
• Improved type safety by replacing !! operator with isDefined utility in SettingsObjectFieldActiveActionDropdown
• This is a breaking change that may affect existing implementations using number fields as record identifiers

_{2 files reviewed, 1 comment
Edit PR Review Bot Settings | Greptile}

[github-actions]

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:20663

This environment will automatically shut down when the PR is closed or after 5 hours.

[prastoin]

@greptileai trigger

[greptile-apps]

PR Summary

Major validation and backend implementation updates for label identifier changes, focusing on ensuring only text-based fields can be used as identifiers.

• Added validateMetadataIdentifierFieldMetadataId utility to enforce label/image identifier validation in src/engine/metadata-modules/utils/validate-metadata-identifier-field-metadata-id.utils.ts
• Implemented unicity row hook in spreadsheetImportGetUnicityRowHook.ts to properly validate unique constraints during data imports
• Added comprehensive error tracking with MetricsService integration across GraphQL operations and workflow executions
• Migrated several database queries from TypeORM to direct SQL via WorkspaceDataSourceService for better performance
• Updated 30+ localization files to reflect new workspace creation and data model terminology

_{118 files reviewed, 5 comments
Edit PR Review Bot Settings | Greptile}