Skip to content

Fix configuration toasts and default services to unlimited resources#132

Merged
arjunkomath merged 3 commits into
mainfrom
fix/deduplicate-configuration-toasts
Jul 11, 2026
Merged

Fix configuration toasts and default services to unlimited resources#132
arjunkomath merged 3 commits into
mainfrom
fix/deduplicate-configuration-toasts

Conversation

@arjunkomath

@arjunkomath arjunkomath commented Jul 11, 2026

Copy link
Copy Markdown
Member

Summary

  • remove duplicate success notifications from serverless, source, and environment-variable updates
  • retain the shared configuration notification and all error feedback
  • default newly created services to no CPU or memory limits
  • stop generating resource limits in tc init manifests
  • preserve existing service limits when a CLI manifest omits resources
  • allow an explicit empty resources: {} block to remove limits
  • retain all opt-in resource limit presets

Existing services keep their configured limits; this only changes defaults for newly created services.

Verification

  • git diff --check
  • go test ./internal/cli ./internal/manifest
  • npx tsc --noEmit
  • npx biome check actions/projects.ts lib/cli-service.ts components/service/details/resource-limits-section.tsx lib/constants.ts db/schema.ts
  • Toast-only files were checked separately; Biome reports a pre-existing import-order issue in secrets-section.tsx

@github-actions

github-actions Bot commented Jul 11, 2026

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit ec58874.

@arjunkomath arjunkomath changed the title Deduplicate service configuration toasts Fix configuration toasts and default services to unlimited resources Jul 11, 2026
@arjunkomath arjunkomath merged commit c280d79 into main Jul 11, 2026
14 checks passed
@arjunkomath arjunkomath deleted the fix/deduplicate-configuration-toasts branch July 11, 2026 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant