feat: serverless worker deployment CRUD#3236
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
9c01c45 to
751339c
Compare
751339c to
44488b5
Compare
44488b5 to
d7f777f
Compare
69ae808 to
750e13f
Compare
740ee5c to
8b91694
Compare
rossnelson
changed the title
|
Looking and working great! Some UI-specific feedback based on the preview URL. Mostly nits. 🙈 Create Worker Deployment + Create Worker Deployment Version pages[1] Page heading should be a simple [2] [3] We can probably remove the hint on the "Name" field. What do you think? I feel it's self-explanatory. [4] Is Build ID required? I thought this was going to be optional. Let's capitalize the W in Worker. [5] The tooltips are being cutoff. Do we feel both tooltips AND helper texts are necessary? Maybe we should only keep one of them? 
[6] Can rename the confirmation button to simply "Create" instead of Create Worker Deployment. [7] Use [8] Use size [9] For compute provider inputs container, the background should be [10] The vertical steppers should match the style in Audit Log Integration setup. They are also used in Export. We should probably componentize them now that this is the 3rd area to utilize them. [11] Btw love what you did with the AWS Setup Guide on smaller screens. The row for [12] Let's use [13] The links in the setup guide don't have a hover state. Should we simply use our Link component here? Deployments table[1] Let's add [2] I realize this is probably happening because of the deployment name length, but is it possible to align the build status badges across rows so they start at the same position? It would be much easier to scan the list that way. Deployments detail page[1] Use [2] Update gap between View Workflows and Create New Version buttons to [3] Use destructive ghost button for Delete in the row action menu. [4] Love Validate Connection modal. [5] When the version row is expanded, we can add the ability to copy Lambda ARN and IAM Role? |
- Remove tooltips from compute form fields, rely on hint text only - Swap AWS console links in setup guide to Link component - Card padding p-4→p-5, section headings font-semibold→font-medium - Scaling toggle button size sm, remove static name hint text - Add bg-blue-50 to setup guide sidebar card - Delete MenuItem uses destructive prop - Copyable wrapper on lambda/IAM ARN in version compute details - radio-card selected state always border-subtle, slot bg surface-background - timeline-step heading text-base font-medium - Extract handleCreate + rollback helpers in serverless-worker-create - Add error handling for setCurrentDeploymentVersion with full rollback
- Use surface-information (theme-aware indigo tint) instead of bg-blue-50 - Apply surface-information to drawer as well as sidebar card - Remove IAM console link from setup guide - Update setup guide copy to reference CloudFormation template specifically
- Empty state: use workers icon with text-blue-200, flex-wrap buttons - version-row-details: $derived.by with retryCount for $derived suggestion - version-table-row: validateLoading=false in onError callback - deployment.svelte: reset deleteError before retry - deployments.ts: narrow isVersionSummaryNew to include non-null deploymentVersion - version-table-row: remove ! non-null assertions (now covered by type guard) - scaling toggle: add chevron-up/down trailingIcon in both create forms - create-version-form: add try/catch in onUpdate
All three forms (create deployment, create version, edit version) shared identical compute field markup. Extracted into ComputeFields with $bindable props. Also removes leftover Tooltip/Icon wrappers from edit-version-form that were inconsistent with the other forms.
- Extract DeleteConfirmModal with shared state, template, and logic; DeleteDeploymentModal and DeleteVersionModal become thin wrappers - Show Create Worker Deployment button on instances tab (mirrors deployments tab)
laurakwhit
left a comment
laurakwhit
left a comment
There was a problem hiding this comment.
One small comment below, but otherwise 🚀
Moves delete-confirm-modal.svelte from the deployments directory to src/lib/components/delete-confirmation-modal.svelte so it can be reused across the app. Updates both delete-deployment-modal and delete-version-modal to import from the new shared location.
98badb5 to
d4a7c90
Compare
Auto-generated version bump from 2.48.3 to 2.48.4 Specific version: 2.48.4 Changes included: - [`7cf1be53`](7cf1be5) Adjust breadcrumb spacing and min-height (#3304) - [`c9834815`](c983481) Add missing Nexus Operation events to event groups (#3309) - [`5c7aa86c`](5c7aa86) feat: add onItemsChange callback to PaginatedTable (#3300) - [`66d3cf68`](66d3cf6) Update pre-release badge to secondary in Workers (#3310) - [`393e84de`](393e84d) Keep Pending Activities/Operations at the top of Timeline and Compact view (#3307) - [`9f721347`](9f72134) Update error link in codec-server-error-banner (#3313) - [`e698bda3`](e698bda) Add class prop to BottomNav component (#3314) - [`55767fc9`](55767fc) Add overrides (#3317) - [`256cd274`](256cd27) refactor(decode-payload): clean up decode-payload.ts API surface (#3302) - [`04f8c251`](04f8c25) Make input height full so its easier to click into the input (#3318) - [`b2231056`](b223105) Workflows Table Row Density (#3285) - [`0b2ce4fd`](0b2ce4f) PR Review Notifications (#3322) - [`84618196`](8461819) feat: serverless worker deployment CRUD (#3236) Co-authored-by: rossnelson <[email protected]>
4 participants
Summary
Implements full CRUD UI for serverless worker deployments under `/workers/deployments/`:
serverless-ui-walkthrough.mov
Also see worker code at: https://github.com/rossnelson/temporal-serverless-worker/blob/main/SETUP.md
Deployments
Versions
Compute Provider
Infrastructure
Test plan