Skip to content

Conversation

dngomez
Copy link
Contributor

@dngomez dngomez commented Oct 3, 2025

Modified the instrument resolver, so every time a configuration is not found, it will try to get the configuration related to the same instrument but using wfs: NONE and create a copy using the requested wfs, the new configuration will have a comment which will be displayed using a toast in the UI to warn the client.
If nothing is found, then an default configuration using default values (0.0 or false in most of them) and the same toast will warn the user with a different message.

Currently if no configuration is found the system doesn't know what to do and it is necessary to manually connect to the database and create a configuration.

@dngomez dngomez requested a review from hugo-vrijswijk October 3, 2025 12:47
@bundlemon
Copy link

bundlemon bot commented Oct 3, 2025

BundleMon

Files updated (1)
Status Path Size Limits
assets/index-(hash).js
559.73KB (+47B +0.01%) -
Unchanged files (2)
Status Path Size Limits
assets/prime-(hash).js
163.5KB -
assets/index-(hash).css
46.93KB -

Total files change +47B +0.01%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

Copy link
Contributor

@hugo-vrijswijk hugo-vrijswijk left a comment

Choose a reason for hiding this comment

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

Minor comments, but overall looks good! Can you also add some tests? There should already be some for the instruments

@hugo-vrijswijk hugo-vrijswijk enabled auto-merge (squash) October 7, 2025 09:00
@hugo-vrijswijk hugo-vrijswijk merged commit bc2f4bb into main Oct 7, 2025
10 checks passed
@hugo-vrijswijk hugo-vrijswijk deleted the sc-6925-create-fallback-instrument-configuration branch October 7, 2025 09:01
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