Skip to content

[Dashboard] Unhide Engine Cloud service #7215

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.

Sign up for GitHub

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

Merged
merged 1 commit into from
May 30, 2025
Merged

[Dashboard] Unhide Engine Cloud service #7215

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/lucky-jobs-relate.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@thirdweb-dev/service-utils": patch
---

Update service descriptions
2 changes: 1 addition & 1 deletion apps/dashboard/src/components/settings/ApiKeys/validations.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -147,4 +147,4 @@ export type ApiKeyPayConfigValidationSchema = z.infer<
>;

// FIXME: Remove
export const HIDDEN_SERVICES = ["relayer", "chainsaw", "engineCloud"];
export const HIDDEN_SERVICES = ["relayer", "chainsaw"];
6 changes: 3 additions & 3 deletions packages/service-utils/src/core/services.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export const SERVICE_DEFINITIONS = {
insight: {
name: "insight",
title: "Insight",
description: "Insight services",
description: "Indexed data for any EVM chain",
// all actions allowed
actions: [],
},
Expand All @@ -76,9 +76,9 @@ export const SERVICE_DEFINITIONS = {
},
engineCloud: {
name: "engineCloud",
title: "Server wallets",
title: "Engine Cloud",
description:
"Server wallets with high transaction throughput and low latency",
"Transaction API and Server wallets with high transaction throughput and low latency",
// all actions allowed
actions: [],
},
Comment on lines 77 to 84
Copy link
Contributor

Choose a reason for hiding this comment

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

💡 Verification agent

🧩 Analysis chain

Verify removal of legacy references to "Server wallets"
The engineCloud title and description have been updated to reflect its new branding. To ensure no stale references remain, search the codebase for the old title "Server wallets" and update any occurrences (docs, UI copy, tests).

🏁 Script executed:

#!/bin/bash
# Verify no leftover references to the old title
rg "Server wallets"

Length of output: 722

Remove all legacy “Server wallets” references
We located several stray “Server wallets” strings that need updating to the new Engine Cloud branding:

  • packages/service-utils/src/core/services.ts (lines 77–84) 
    - description: "Transaction API and Server wallets with high transaction throughput and low latency",
+ description: "Transaction API and Engine Cloud wallets with high transaction throughput and low latency",
  • apps/portal/src/app/engine/v2/configure-wallets/circle-wallet/page.mdx
    Replace “Server wallets” with “Engine Cloud wallets” (or your preferred terminology).
  • apps/portal/src/app/engine/v3/get-started/page.mdx
    Update both the explanatory copy and the internal link text/URL to reflect Engine Cloud.
  • apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/engine/cloud/analytics/ftux.client.tsx
    Change the description to remove “Server wallets” in favor of the new branding.

Please apply these replacements so that all references align with the Engine Cloud terminology.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
engineCloud: {
name: "engineCloud",
title: "Server wallets",
title: "Engine Cloud",
description:
"Server wallets with high transaction throughput and low latency",
"Transaction API and Server wallets with high transaction throughput and low latency",
// all actions allowed
actions: [],
},
engineCloud: {
name: "engineCloud",
title: "Engine Cloud",
description:
"Transaction API and Engine Cloud wallets with high transaction throughput and low latency",
// all actions allowed
actions: [],
},
🤖 Prompt for AI Agents 
In packages/service-utils/src/core/services.ts around lines 77 to 84, update the
title and description fields to remove any legacy references to "Server wallets"
and replace them with the new branding "Engine Cloud" or "Engine Cloud wallets"
as appropriate. Additionally, search the entire codebase for all occurrences of
"Server wallets" and update them consistently to reflect the new Engine Cloud
terminology, including in documentation, UI copy, tests, and configuration files
as specified in the review comment.

Expand Down
Loading