Enhance error message for duplicated id #173
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Improved the error message when attempting to open multiple overlays with the same
overlayId.Related Issue: Fixes #172
Why is this change required?
The original error message did not indicate which
overlayIdwas causing the conflict, making it difficult to debug in complex UI scenarios.What problem does it solve?
Developers can now immediately see which
overlayIdwas duplicated, reducing debugging time.Dependencies: None
Changes
Updated the error message in the overlay reducer to include the conflicting
overlayId:Motivation and Context
This change was motivated by recurring developer confusion when encountering the "duplicate overlayId" error without enough context. Including the
overlayIddirectly in the error message makes it immediately actionable and improves DX .How Has This Been Tested?
None
Screenshots (if appropriate):
Not applicable
Types of changes
Checklist
Further Comments
None.