Skip to content

fix: improve error message for schema problems W-20495414 - #1363

Merged
jfeingold35 merged 10 commits into
mainfrom
sm/qa-fixes
Dec 18, 2025
Merged

fix: improve error message for schema problems W-20495414#1363
jfeingold35 merged 10 commits into
mainfrom
sm/qa-fixes

Conversation

@mshanemc

Copy link
Copy Markdown
Contributor

What does this PR do?

see #1362 (comment)

What issues does this PR fix or reference?

@W-20495414@

@mshanemc
mshanemc requested a review from a team as a code ownerDecember 18, 2025 16:14
Comment threadsrc/api/data/tree/importPlan.ts Outdated
const parseResults = DataImportPlanArraySchema.safeParse(planContents);

if (parseResults.error) {
// console.log(parseResults);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this be removed?

@jfeingold35
jfeingold35 merged commit 54add09 into mainDec 18, 2025
64 of 65 checks passed
@jfeingold35
jfeingold35 deleted the sm/qa-fixes branch December 18, 2025 17:25
Sign up for freeto 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

@mshanemc@jfeingold35