Co 3770 improve cos module code stability#1211
Draft
giuliano176 wants to merge 257 commits into
Draft
Conversation
Remove unused del dependency from ui-components package.json and update pnpm-lock.yaml to reflect the change.
Bump vite-plugin-svgr from 4.5.0 to 5.2.0. Update pnpm-lock.yaml to reflect new version and dependencies.
Update zustand from 4.x to 5.0.13 in all apps and packages. Adjust pnpm-lock.yaml and dependency-updates.md to reflect the new version.
…bootstrap][cos][dashboard][domains][legalhold][mta][notifications][operations][privacy][storage][subscription][test-utils][ui-shared][root] (pnpm-lock) Update i18next to v26.2.0 and react-i18next to v17.0.8 across all apps and packages. Remove deprecated from i18next type declarations. Update TFunction usage and related types for compatibility with new i18next versions. Rename i18n-test-factory.jsx to .tsx and add explicit return type. Remove node-fetch polyfill from vitest-jsdom-setup as it is no longer needed. Update pnpm-lock.yaml for all dependency changes.
Add AGPL-3.0-only license header for all .md files in the dep5 REUSE specification.
…p][bootstrap][cos][dashboard][domains][legalhold][mta][notifications][operations][privacy][storage][subscription] (i18next) Remove duplicated i18next type augmentation files from all admin-ui apps.
- Move pnpm hoisting and dedupe config from .npmrc to pnpm-workspace.yaml - Update pnpm version to 11.2.2 in package.json - Remove pnpm from dependencies in multiple app and package manifests - Update posthog-js and related dependencies to 1.375.0 - Update msw mockServiceWorker.js to 2.14.6 across all apps and packages - Update semver to 7.8.1 in lockfile - Add libc fields for native packages in lockfile
Remove @vitejs/plugin-react and vite from devDependencies in ui-shared/package.json. Update pnpm-lock.yaml to reflect dependency cleanup.
Bump @tsconfig/vite-react from 7.0.2 to 8.0.6 and enable strict mode in tsconfig.base.json. Update pnpm-lock.yaml accordingly.
Update react and react-dom dependencies to version 19.2.3 across all apps and packages. Update pnpm-lock.yaml accordingly.
Add babel-plugin-react-compiler to the Vite and Vitest configs for admin-ui-bootstrap. Update package.json and pnpm-lock.yaml accordingly.
Remove useMemo and useCallback hooks from app, license-banner, activate-subscription, and subscription components. Simplify logic by using direct assignments and inline functions.
…media-query) - Switch use-media-query hook to useSyncExternalStore API - Bump eslint-plugin-react-hooks to 7.1.1 and enable new rules - Update eslint config and pnpm-lock.yaml accordingly
Fix logic to trigger modifyCosMutation only when cn is unchanged and renameCosMutation when cn is changed.
Remove default export and use named export for FeatureSwitchField.
Add role="switch", aria-checked, aria-label, and aria-disabled to Switch component for better accessibility support.
Extract untrusted network, grace period, and grace period end date picker sections into separate components for improved clarity and maintainability.
…od-end-date-picker) Move Padding and Row components inside GracePeriodEndDatePicker to avoid extra nesting in TwoFactorSection. Updates layout structure for the date picker section.
…al-information.browser.test, app-view.browser.test) Eliminate mockSvgrImageImport usage from COS browser tests. Add default handler for SVG image import requests in test-utils worker.
…count-quota-inputs-new.browser.test) Change quota switch enable/disable assertions to check aria-disabled attribute instead of tabindex in EditAccountQuotaInputsNew tests.
giuliano176
force-pushed
the
CO-3770-improve-cos-module-code-stability
branch
from
cf1f5ce to
6c93c24
Compare
…rm, features-form, general-information-form, preferences-form) Switch dirty state selectors from isDirty to !isDefaultValue in COS forms and privacy view. Update related tests for switch role and input clearing.
…formation-form) Replace duplicated domain/account table logic with new SearchableTable component. Add searchable-table file and update usage in general information form.
…formation-form) Move COS delete modal logic to DeleteCosModal component. Update general information form to use new modal for deletion confirmation.
…on-form) Move attribute-to-map logic to attributesToMap util. Replace duplicate mapping code in general information form and COS data handling.
…ation-form) Move status color/label mapping to buildStatusColorMap helper. Replace inline STATUS_COLOR object with function usage in general information form.
…lds, general-information-form) Move COS general info form fields to CosInfoFields component. Update GeneralInformationForm to use new component for rendering info fields.
…sword, cos-list-panel, cos-list, general-information-form, searchable-table) Replace hardcoded color values with design tokens and semantic color names for backgrounds, borders, and text in COS views and tables.
Remove FC typing from SearchableTable and use explicit props type annotation. Adjusts export and props handling for consistency.
Reorganize container structure for correct layout rendering and update server filtering logic for clarity. Remove unnecessary FC typing and adjust button placement and props for enable/disable actions.
Move the disable pool confirmation modal to its own component (disable-pool-modal) and update usage in cos-server-pools for clarity and maintainability.
Move server pool table UI to a new ServerPoolTable component for separation of concerns and simplify cos-server-pools.
Replace manual debounce logic with useDebouncedValue hook for server search input.
Move row click logic to table row props and rename onModifyCOS to onModifyCos for consistency.
Relocated CosServerPools and related components (funnel-search-icon, disable-pool-modal, server-pool-table) into a dedicated cos-server-pools subdirectory. Updated all imports and tests to reflect the new structure.
Moved CosFeatures, FeaturesForm, and related test files into a new cos-features subdirectory. Updated all relevant imports and test references accordingly.
Refactored status display logic in cos-list to use a config object for color and label. Improved empty and error state handling, debounced search and resize, and cleaned up code for clarity.
Removed default exports from create-new-cos and general-list-panel views. Updated related imports in tests to use named exports.
Moved cos-list to a dedicated cos-list subfolder and updated all related imports to reflect the new path.
Replaced row click selection with onSelectionChange handler in cos-server-pools and server-pool-table. Updated table props and selection logic accordingly.
…ail-panel-routing.test) Updated vi.mock paths in cos-detail-panel-routing.test to match new cos-features and cos-server-pools subfolder structure.
…on (cos-server-pools)
|
0 New Issues
4 Fixed Issues
0 Accepted Issues
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.