Skip to content

refactor(project-settings): use shared Select in GitHub settings modal#10360

Open
juarezsousa-ctrl wants to merge 1 commit intohighlight:mainfrom
juarezsousa-ctrl:fix/antd-single-settings-page
Open

refactor(project-settings): use shared Select in GitHub settings modal#10360
juarezsousa-ctrl wants to merge 1 commit intohighlight:mainfrom
juarezsousa-ctrl:fix/antd-single-settings-page

Conversation

@juarezsousa-ctrl
Copy link
Copy Markdown

@juarezsousa-ctrl juarezsousa-ctrl commented Mar 31, 2026

PR Title

/claim #8635
Closes #8635

refactor(project-settings): use shared Select in GitHub settings modal

PR Description

Summary
This updates the Project Settings GitHub settings modal to use the repository’s shared Select component instead of importing Select directly from antd.

Changes

Replaced direct antd Select usage in frontend/src/pages/ProjectSettings/GitHubSettingsModal/GitHubSettingsModal.tsx
Updated the options shape from label to displayValue to match the shared wrapper API
Kept modal behavior and existing value/filtering logic unchanged
Testing

Reviewed the shared Select wrapper API and confirmed it accepts the props used in this modal
Confirmed the wrapper expects displayValue and internally maps it to Ant Design label
Compared this usage against existing repository patterns in nearby settings pages
Local app-wide validation was limited by workspace bootstrap/build dependencies in other local packages, but the frontend-specific path was brought far enough to verify that the original @highlight-run/react import-resolution issue was unrelated to this change
Scope

Scoped to a single file only:
frontend/src/pages/ProjectSettings/GitHubSettingsModal/GitHubSettingsModal.tsx
This change follows existing repository patterns for shared UI abstractions and keeps the modal behavior intact.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 31, 2026

⚠️ No Changeset found

Latest commit: 973cf27

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@juarezsousa-ctrl
Copy link
Copy Markdown
Author

Scoped this to a single settings modal to keep the change small and reviewable.

This replaces the direct antd Select import with the shared Select wrapper already used in nearby settings pages, and updates the options shape to match the wrapper API (displayValue).

I did not add a changeset because this is a small internal refactor with no intended user-facing versioned change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update workspace and project settings to not use antd components

1 participant