feat(GitLab): Link GitLab issues and merge requests to feature flags#7274
feat(GitLab): Link GitLab issues and merge requests to feature flags#7274emyller wants to merge 3 commits intofeat/gitlab-browse-frontendfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
4a25c7e to
8c91bf3
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## feat/gitlab-browse-frontend #7274 +/- ##
==============================================================
Coverage ? 98.36%
==============================================================
Files ? 1366
Lines ? 51154
Branches ? 0
==============================================================
Hits ? 50320
Misses ? 834
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
8c91bf3 to
a214ee7
Compare
There was a problem hiding this comment.
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.
Docker builds report
|
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
|
Visual Regression16 screenshots compared. See report for details. |
| "issue-linked", | ||
| project__id=feature.project_id, |
There was a problem hiding this comment.
8bf6b03 — also fixed other GitLab logs since we're here.
Zaimwa9
left a comment
There was a problem hiding this comment.
Couple of mostly minor comments that should be quite straightforward 👍
| ) -> Response: | ||
| response = super().create(request, *args, **kwargs) | ||
| structlog.get_logger("gitlab").info( | ||
| "issue-linked", |
There was a problem hiding this comment.
Good to see but actually the naming should follow entity.action https://github.com/Flagsmith/flagsmith/pull/7268/changes#diff-c7b1c23e00a1fd0378f6ea41a7dfe1c9e043d766f738dba26860c783b82e8c6aR91
Same below
There was a problem hiding this comment.
Should we add organisation Id ?
There was a problem hiding this comment.
Agreed, thanks for pointing to guidance docs. 8bf6b03 — also fixed other recently added logs to meet guidelines.
| metadata: { | ||
| state: selectedItem.state, | ||
| title: selectedItem.title, | ||
| ...('merged' in selectedItem && { merged: selectedItem.merged }), | ||
| ...('draft' in selectedItem && { draft: selectedItem.draft }), |
There was a problem hiding this comment.
We are never passing iids ? cf backend related test
| }).then(() => { | ||
| toast('GitLab link added') | ||
| setSelectedItem(null) | ||
| AppActions.refreshFeatures(projectId, environmentId) | ||
| }) |
There was a problem hiding this comment.
Same we'd need to catch error (and at the same time move to async/await 😄 )
| *args: Any, | ||
| **kwargs: Any, | ||
| ) -> Response: | ||
| response = super().create(request, *args, **kwargs) |
There was a problem hiding this comment.
It seems the payload has already been validated, so shouldn't it be a perform_create directly ?
a214ee7 to
8bf6b03
Compare
I have added information todocs/if required so people know about the feature.Changes
Closes #7160
Stack: #7270 (backend) → #7273 (frontend) → #7274 (this)
With browsing in place (#7270, #7273), users can now link GitLab issues and merge requests to their feature flags. This completes the core linking flow.
Slight refactor:
<ExternalResourcesTable>now renders regardless of GitHub.Review effort: 2/5
How did you test this code?
Integration tests. Manual UI tests. Screenshots: