Skip to content

Improve message handling in VR logger by explicitly mapping null and undefined values#9

Merged
MakingSpiderSense merged 1 commit into
mainfrom
bugfix/vr-logger/93-fix-type-error
Aug 19, 2025
Merged

Improve message handling in VR logger by explicitly mapping null and undefined values#9
MakingSpiderSense merged 1 commit into
mainfrom
bugfix/vr-logger/93-fix-type-error

Conversation

@MakingSpiderSense
Copy link
Copy Markdown
Owner

This pull request makes a small change to the vr-logger component to improve how console messages are logged. The update ensures that null and undefined values are explicitly logged as strings, rather than as empty strings or causing errors.

https://trello.com/c/yhfzY979

Copilot AI review requested due to automatic review settings August 19, 2025 21:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the VR logger component's message handling by explicitly converting null and undefined values to their string representations instead of relying on the default toString() behavior.

  • Enhanced null/undefined handling in console message logging
  • Replaced single-line mapping with explicit conditional checks for better clarity

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/components/vr-logger/vr-logger.js
Comment thread src/components/vr-logger/vr-logger.js
@MakingSpiderSense MakingSpiderSense merged commit 51853d3 into main Aug 19, 2025
@MakingSpiderSense MakingSpiderSense deleted the bugfix/vr-logger/93-fix-type-error branch August 19, 2025 21:08
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