-
Notifications
You must be signed in to change notification settings - Fork 27
feat(services): add git webhook status badge (QOV-1504) #2331
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(services): add git webhook status badge (QOV-1504) #2331
Conversation
|
Qovery can create a Preview Environment for this PR.
This comment has been generated from Qovery AI 🤖.
|
554108f to
9686983
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## staging #2331 +/- ##
===========================================
+ Coverage 47.53% 47.78% +0.25%
===========================================
Files 1279 1267 -12
Lines 23524 23070 -454
Branches 6931 6781 -150
===========================================
- Hits 11182 11025 -157
+ Misses 10191 9945 -246
+ Partials 2151 2100 -51
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
17ff31c to
04f9dab
Compare
04f9dab to
4ee954d
Compare
4ee954d to
6b504a2
Compare
| }), | ||
| }) | ||
|
|
||
| export type { GitWebhookStatusResponse } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fabienfleureau Get rid of this export please 🙏
| @@ -0,0 +1 @@ | |||
| export * from './auto-deploy-section' | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you get rid of this file?
| </Section> | ||
| </> | ||
| )) | ||
| .with({ serviceType: 'CONTAINER' }, () => ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| .with({ serviceType: 'CONTAINER' }, (container) => ( |
| <Heading>Deploy</Heading> | ||
| <EntrypointCmdInputs /> | ||
| <AutoDeploySetting source="CONTAINER_REGISTRY" /> | ||
| <AutoDeploySection serviceId={service.id} source="CONTAINER_REGISTRY" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| <AutoDeploySection serviceId={service.id} source="CONTAINER_REGISTRY" /> | |
| <AutoDeploySection serviceId={container.id} source="CONTAINER_REGISTRY" /> |
Summary
Issue: QOV-1504 QOV-1499
Adds a git webhook status badge to service settings pages, showing whether auto-deploy webhooks are properly configured.
Adds a button to set the webhook on git provider (visible only when needed)
Changes
GitWebhookStatusBadgecomponent with states: Active, Not Configured, Misconfigured, Unable to VerifyuseGitWebhookStatushook wrapping the API callScreenshots / Recordings
Testing
yarn testoryarn test -u(if you need to regenerate snapshots)yarn formatyarn lintPR Checklist
.cursor/rules)feat(service): add new Terraform service) - required for semantic-release