Skip to content

Remove manual support triage fields#3037

Merged
ChiragAgg5k merged 3 commits intomainfrom
simplify-support-triage-form
May 11, 2026
Merged

Remove manual support triage fields#3037
ChiragAgg5k merged 3 commits intomainfrom
simplify-support-triage-form

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

Summary

  • remove manual category, topic, and severity controls from the support wizard
  • submit only ticket content and account/project context so backend triage is authoritative
  • simplify support wizard state shape

Testing

  • bunx prettier --check src/routes/(console)/supportWizard.svelte src/routes/(console)/wizard/support/store.ts
  • bunx eslint src/routes/(console)/supportWizard.svelte src/routes/(console)/wizard/support/store.ts
  • bun run check (fails on existing unrelated SDK/model type errors; no diagnostics for changed support files when filtered)

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 10, 2026

Greptile Summary

This PR removes the manual category, topic, and severity controls from the support wizard, delegating triage entirely to the backend. The store shape is simplified by dropping category and topic, and message/subject are now nullable.

  • The SupportData type drops category and topic, and relaxes message/subject to string | null so the store can represent a truly empty initial state.
  • handleSubmit uses ?? '' to safely coerce null values before appending to FormData, and no longer constructs or sends the categoryTopicTag.
  • The onDestroy reset omits the project key (while resetData includes project: null), leaving the property as undefined after unmount — harmless because project is optional and the ?? '' fallback in handleSubmit covers undefined too.

Confidence Score: 5/5

Safe to merge — the change removes UI-only fields and does not alter the core submission flow in any breaking way.

The diff is a straightforward deletion of category/topic fields with no new logic introduced. Null coalescing guards are in place for the nullable fields before they reach FormData, and the required form validation prevents empty submissions. No data is dropped that the backend depended on (it now owns triage), and the cleanup is symmetric across the store type, component state, and submit handler.

No files require special attention.

Important Files Changed

Filename Overview
src/routes/(console)/supportWizard.svelte Removes category/topic tag UI, drops the derived categoryTopicTag from FormData, and guards null subject/message with nullish coalescing on FormData.append
src/routes/(console)/wizard/support/store.ts Drops category and topic fields from SupportData type; changes message/subject initial values from empty strings to null; project becomes string

Reviews (3): Last reviewed commit: "Merge origin/main into simplify-support-..." | Re-trigger Greptile

Comment thread src/routes/(console)/wizard/support/store.ts
@ChiragAgg5k ChiragAgg5k merged commit 6eb14b6 into main May 11, 2026
3 of 4 checks passed
@ChiragAgg5k ChiragAgg5k deleted the simplify-support-triage-form branch May 11, 2026 06:49
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