-
Notifications
You must be signed in to change notification settings - Fork 27
feat(services): add AutoDeployBadge component as header for services with git source QOV-1504 #2338
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
base: staging
Are you sure you want to change the base?
Conversation
|
Qovery can create a Preview Environment for this PR.
This comment has been generated from Qovery AI 🤖.
|
…with git source (QOV-1504)
ee5c4e4 to
30d4c8d
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## staging #2338 +/- ##
===========================================
+ Coverage 47.78% 49.41% +1.62%
===========================================
Files 1267 691 -576
Lines 23070 15000 -8070
Branches 6781 4487 -2294
===========================================
- Hits 11025 7412 -3613
+ Misses 9945 6194 -3751
+ Partials 2100 1394 -706
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:
|
rmnbrd
left a comment
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.
| config.color === 'green' | ||
| ? 'text-green-500' | ||
| : config.color === 'yellow' | ||
| ? 'text-yellow-500' | ||
| : config.color === 'red' | ||
| ? 'text-red-500' | ||
| : 'text-neutral-350' |
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 Could you please directly put into the webhookStatusConfig object the color-related classname instead of having this condition?

Summary
Issue: QOV-1504
User can directly see auto-deploy status on header of their services
There is also the webhook status
Screenshots / 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