Skip to content

feat(gitlab): add frontend components for GitLab integration#7123

Open
asaphko wants to merge 2 commits intofeature/gitlab-webhookfrom
feature/gitlab-frontend
Open

feat(gitlab): add frontend components for GitLab integration#7123
asaphko wants to merge 2 commits intofeature/gitlab-webhookfrom
feature/gitlab-frontend

Conversation

@asaphko
Copy link
Copy Markdown
Contributor

@asaphko asaphko commented Apr 3, 2026

Add all frontend components for the GitLab integration. Final PR in the stacked series splitting #7028.

Changes

  • RTK Query services: useGitlabIntegration.ts (CRUD), useGitlab.ts (projects, resources)
  • GitLabSetupPage.tsx — data-fetching container, delegates to sub-components
  • GitLabIntegrationDetails.tsx — connected state: repo selection, tagging toggle, webhook config
  • CreateGitLabIntegrationForm.tsx — credentials input form
  • GitLabResourcesSelect.tsx — issue/MR search for Links tab
  • useHasGitlabIntegration hook
  • Types: request/response types for all GitLab endpoints
  • Constants: GitLab resource types, default integration flags
  • Modified: IntegrationList, ExternalResourcesLinkTab, ExternalResourcesTable, create-feature modal

Contributes to #7028
Stacks on #7122

Review effort: 3/5

Review feedback addressed from #7028

  • Extracted connected-state block into GitLabIntegrationDetails component (Zaimwa9)
  • Extracted setup form into CreateGitLabIntegrationForm component (Zaimwa9)
  • GitLabSetupPage focused on data handling only (Zaimwa9)
  • async/await in handleLinkRepo and handleDelete (Zaimwa9)

🤖 Generated with Claude Code

@asaphko @claude
feat(gitlab): add frontend components for GitLab integration
72eaba4 
Add RTK Query services, setup page, resource selector, and integration
list support for the GitLab integration. Setup page split into three
components per review feedback.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@asaphko asaphko requested a review from a team as a code owner April 3, 2026 13:43
@asaphko asaphko requested review from Zaimwa9 and removed request for a team April 3, 2026 13:43
claude[bot]
claude bot reviewed Apr 3, 2026
View reviewed changes
Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, reopen this pull request to trigger a review.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 3, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
flagsmith-frontend-preview Ready Ready Preview, Comment Apr 3, 2026 1:45pm
flagsmith-frontend-staging Ready Ready Preview, Comment Apr 3, 2026 1:45pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview Apr 3, 2026 1:45pm

Request Review

@github-actions github-actions bot added the front-end Issue related to the React Front End Dashboard label Apr 3, 2026
@pre-commit-ci pre-commit-ci bot requested a review from a team as a code owner April 3, 2026 13:44
@github-actions github-actions bot added the feature New feature or request label Apr 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026 •
edited
Loading

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-e2e:pr-7123 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-api-test:pr-7123 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-api:pr-7123 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith:pr-7123 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-private-cloud:pr-7123 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-frontend:pr-7123 Finished ✅ Results ✅

@github-actions github-actions bot added the api Issue related to the REST API label Apr 3, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.10%. Comparing base (bb36a84) to head (ddb97e1).
⚠️ Report is 1 commits behind head on feature/gitlab-webhook.

Additional details and impacted files 
@@                    Coverage Diff                     @@
##           feature/gitlab-webhook    #7123      +/-   ##
==========================================================
- Coverage                   98.10%   98.10%   -0.01%     
==========================================================
  Files                        1360     1360              
  Lines                       50686    50685       -1     
==========================================================
- Hits                        49725    49724       -1     
  Misses                        961      961

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026 •
edited
Loading

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  10 passed

Details

stats  10 tests across 7 suites
duration  43 seconds
commit  72eaba4
info  🔄 Run: #15698 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  10 passed

Details

stats  10 tests across 7 suites
duration  42.7 seconds
commit  ddb97e1
info  🔄 Run: #15699 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  10 passed

Details

stats  10 tests across 7 suites
duration  45 seconds
commit  ddb97e1
info  🔄 Run: #15699 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  10 passed

Details

stats  10 tests across 7 suites
duration  45.6 seconds
commit  72eaba4
info  🔄 Run: #15698 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  16 passed

Details

stats  16 tests across 13 suites
duration  57.5 seconds
commit  72eaba4
info  🔄 Run: #15698 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  53 seconds
commit  72eaba4
info  🔄 Run: #15698 (attempt 1)

This was referenced Apr 3, 2026
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@claude claude[bot] claude[bot] left review comments

@Zaimwa9 Zaimwa9 Awaiting requested review from Zaimwa9 Zaimwa9 is a code owner automatically assigned from Flagsmith/flagsmith-front-end

Assignees

No one assigned

Labels

api Issue related to the REST API feature New feature or request front-end Issue related to the React Front End Dashboard

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@asaphko