fix: toggle settings feature alignment#8484
Conversation
|
Windows and Mac build successful in Unity Cloud! You can find a link to the downloadable artifact below. |
|
Claude finished @lorenzo-ranciaffi's task in 1m 52s —— View job Code Review
Root-Cause AssessmentProblem: This PR fixes the root cause — not a symptom. Adding explicit integer literals to all three enums ( Correctness VerificationI cross-referenced every
Selected spot-checks:
No Blocking Issues FoundThe protective comment added above each enum is appropriate and correctly documents the invariant future contributors must respect. REVIEW_RESULT: PASS ✅ |
|
QA Pass ✅ Tested on Verified:
No regressions found. Logs are clean of any errors related to this fix. EvidenceMac27.04.2026_12.38.52_REC.PR-8484.mac.mp4Windows27.04.2026_12.43.33_REC.mp4 |
Pull Request Description
Fixes #8424
What does this PR change?
This PR re-aligns all the toggle feature values to the proper configuration (every toggle was shifted by 1 by the improperly ordering of
CHAT_REACTIONS_ENABLED_FEATUREon its creation).Now all features (toggle, slider and dropdown) have a fixed index enum value so that every new addition can't affect the previous ones.
Test Steps
Quality Checklist
Code Review Reference
Please review our Branch & PR Standards before submitting. It explains the automated review flow, QA/DEV approval requirements, and what each label does — especially useful for first-time contributors.