-
Notifications
You must be signed in to change notification settings - Fork 544
[Dashboard] Disable engineCloud service by default #6530
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
[Dashboard] Disable engineCloud service by default #6530
Conversation
)" This reverts commit 21389bb.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
7fcb0ed
to
043d23c
Compare
f3b889f
to
9a538a3
Compare
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6530 +/- ##
=======================================
Coverage 54.92% 54.92%
=======================================
Files 881 881
Lines 55179 55179
Branches 3768 3768
=======================================
Hits 30309 30309
Misses 24775 24775
Partials 95 95
🚀 New features to boost your workflow:
|
PR-Codex overview
This PR focuses on updating the handling of hidden services in the
ApiKeys
component and modifying the service filtering logic in theCreate
component to include an additional service.Detailed summary
validations.ts
,HIDDEN_SERVICES
is updated to include"engineCloud"
.index.tsx
, the filtering logic forALL_PROJECT_SERVICES
is modified to exclude"engineCloud"
as well, with a comment indicating it will be enabled once the API server is ready.