Skip to content

unify schema-driven settings forms #2510

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
juliusmarminge wants to merge 2 commits into
base: main
Choose a base branch
from

Delete apps/web/src/components/settings/MainSettingsForm.test.ts

a02ed3d
Select commit
Loading
Failed to load commit list.
Open

unify schema-driven settings forms #2510

Delete apps/web/src/components/settings/MainSettingsForm.test.ts
a02ed3d
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Approvability Check completed May 5, 2026 in 5m 8s

Not eligible (click on check for details)

  • Eligibility
  • Correctness

Note

More information about how approvability works can be found in our Help Center.

Details

Eligibility
This refactoring PR has an unresolved high-severity review comment identifying a potential bug where settings fields may not render due to AST inference failing on schema-wrapped types. This substantive concern requires human verification.

Notes:
No code owners found in target merge base 91a03e0

View more details on MacroscopeApp