Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(browser-repl): account for node-runtime-worker-thread-specific result rendering MONGOSH-1323 #1452

Merged
merged 1 commit into from
Apr 25, 2023

Conversation

addaleax
Copy link
Contributor

d1278ec attempted to address MONGOSH-1323 but failed to do so properly because node-runtime-worker-thread adjusts behavior so that most values result in reporting a string result back to the frontend.

…sult rendering MONGOSH-1323

d1278ec attempted to address MONGOSH-1323 but failed to do so
properly because node-runtime-worker-thread adjusts behavior so that
most values result in reporting a `string` result back to the frontend.
@addaleax addaleax marked this pull request as ready for review April 25, 2023 12:50
Copy link
Contributor

@gribnoysup gribnoysup left a comment

Choose a reason for hiding this comment

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

That was a quick fix! And thanks for adding a test for that!

@addaleax addaleax merged commit fdc4c1f into main Apr 25, 2023
40 of 46 checks passed
@addaleax addaleax deleted the 1323-dev-2 branch April 25, 2023 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants