Skip to content

fix(squads): create a Startups company without a description - #7290

Merged
idoshamun merged 1 commit into
mainfrom
fix/startups-company-empty-description
Nov 8, 2026
Merged

idoshamun merged 1 commit into
mainfrom
fix/startups-company-empty-description

Conversation

@idoshamun

@idoshamun idoshamun commented Nov 8, 2026 •

Copy link
Copy Markdown
Member

Requested by Ido · project thread

Changes

Found in the end-to-end test of the daily.dev for Startups checkout: on the Company step, Continue did nothing.

  • The step creates the squad with description: ''. The API validates description against ^[\S\s]{1,250}$, so it answers {"description":"description is invalid!"} and no squad is created.
  • createSquad now leaves an empty description out of the CreateSquad variables. Creating a squad with a description is unchanged, and so is editing.
  • useSquadCreate showed result.handle as the toast for any validation error, so this failure showed an empty toast. It now falls back to the default error message when the error isn't about the handle.

Events

No.

Experiment

No.

Manual Testing

Ran typecheck and lint on the changed files, plus the squad and checkout Jest suites (8 suites, 35 tests pass). The only typecheck error is in ArchiveFeedPage.spec.tsx and was already on main. After deploy, the e2e run resumes from the Company step on production.


Generated by Claude Code

Preview domain

https://fix-startups-company-empty-descr.preview.app.daily.dev

The company step sends description '', which the API rejects
(description is invalid), so Continue did nothing. Leave an empty
description out of CreateSquad, and show the default error toast when
the API error isn't about the handle.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FodmVu511cm42Xwnsvau3u
@idoshamun idoshamun self-assigned this Nov 8, 2026
@vercel

vercel Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
daily-webapp Ready Ready Preview Nov 8, 2026 8:20am UTC
storybook Ready Ready Preview Nov 8, 2026 8:20am UTC

Request Review

@idoshamun
idoshamun enabled auto-merge (squash) November 8, 2026 08:20
@idoshamun
idoshamun merged commit 46ba66b into main Nov 8, 2026
13 checks passed
@idoshamun
idoshamun deleted the fix/startups-company-empty-description branch November 8, 2026 08:21

This branch was successfully deployed

2 active deployments
Preview – daily-webapp — 0670fad0 Deployed Nov 8, 2026 by vercel[bot]
Preview – storybook — 0670fad0 Deployed Nov 8, 2026 by vercel[bot]
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.

2 participants