Skip to content

Conversation

@Lord-Grey
Copy link
Collaborator

Summary

Fix - Correct JSON processing
Fix - Show effectsconfigurator menu entry, only if effectengine is available

Improvements

  • JSON getconfig API - allow filter to exclude elements via "null"
  • Update schemas to accept empty arrays
  • Stop WebServer before JSON API to avoid blocking calls by UI

hyperion.ng/commit/dbfc59a28c09ff3bd91fca81d588c1d0a1bba725)

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • [] Docs
  • Build-related changes
  • Other, please describe:

If changing the UI of web configuration, please provide the before/after screenshot:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing setups:

The PR fulfills these requirements:

  • When resolving a specific issue, it's referenced in the PR's body (e.g. Fixes: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature
  • Related documents have been updated (docs/docs/en)
  • Related tests have been updated

PLEASE DON'T FORGET TO ADD YOUR CHANGES TO CHANGELOG.MD

  • Yes, CHANGELOG.md is also updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

@Lord-Grey Lord-Grey self-assigned this Apr 18, 2025
@Lord-Grey Lord-Grey added this to Web UI and Core Apr 18, 2025
@Lord-Grey Lord-Grey moved this to In Progress in Core Apr 18, 2025
@Lord-Grey Lord-Grey moved this to In Progress in Web UI Apr 18, 2025
@Lord-Grey Lord-Grey changed the title Migration Fixing Configuration mMigration issues Apr 18, 2025
@Lord-Grey Lord-Grey changed the title Fixing Configuration mMigration issues Fixing configuration migration issues Apr 18, 2025
@Lord-Grey Lord-Grey merged commit 6f4d58c into hyperion-project:master Apr 18, 2025
20 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Core Apr 18, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Web UI Apr 18, 2025
@Lord-Grey Lord-Grey deleted the Migration branch April 18, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant