Skip to content

fix(preset-maker): validate presets before export#8

Merged
Microck merged 1 commit into
mainfrom
fix/preset-maker-validation
Apr 10, 2026
Merged

fix(preset-maker): validate presets before export#8
Microck merged 1 commit into
mainfrom
fix/preset-maker-validation

Conversation

@Microck
Copy link
Copy Markdown
Owner

@Microck Microck commented Apr 10, 2026

Summary

  • adds visible multi-preset controls to the preset maker UI
  • blocks invalid exports when preset IDs are missing or duplicated
  • validates custom day values so out-of-range values do not generate broken JSON
  • prevents invalid downloads from writing a config file

Verification

  • verified locally through browser automation on the patched page
  • confirmed single-preset generation still works
  • confirmed multi-preset generation works from the new UI
  • confirmed duplicate preset IDs are rejected
  • confirmed invalid custom day values like 31 and -2 are rejected
  • confirmed Copy still works
  • confirmed Download JSON writes the expected file for valid input and does not write one for invalid input

Notes

  • left unrelated untracked frontend.md out of the commit

Add explicit multi-preset controls to the preset maker UI so the shipped page matches the generator's multi-preset support.

Reject missing or duplicate preset IDs and invalid custom day values before generating or downloading JSON, and surface those errors inline instead of silently emitting broken output.
@Microck Microck merged commit ce000eb into main Apr 10, 2026
1 check passed
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