-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Version 6 process is not defined #4564
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
Labels
Comments
heath-freenome
added a commit
that referenced
this issue
Apr 17, 2025
Fixed #4564 to eliminate typing issue
8 tasks
@Janrupf fixing this now |
heath-freenome
added a commit
that referenced
this issue
Apr 17, 2025
zfarhad
pushed a commit
to zfarhad/react-jsonschema-form
that referenced
this issue
May 9, 2025
* Fixed rjsf-team#4564 by checking process type Fixed rjsf-team#4564 to eliminate typing issue * - Removed the `indexBrowser` implementation since the typechecking handles it much more simply
zfarhad
pushed a commit
to zfarhad/react-jsonschema-form
that referenced
this issue
May 9, 2025
* Fixed rjsf-team#4564 by checking process type Fixed rjsf-team#4564 to eliminate typing issue * - Removed the `indexBrowser` implementation since the typechecking handles it much more simply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Prerequisites
What theme are you using?
other
Version
6
Current Behavior
process
is referenced in potential browser code:react-jsonschema-form/packages/utils/src/getTestIds.ts
Line 30 in 91ac453
This leads to a javascript exception
process is not defined
Expected Behavior
Properly detect whether
process
is defined or not. Just referencing it may throw an error in browser environments.Steps To Reproduce
Environment
Anything else?
My suggestion would be to wrap this in a
typeof(process) !== "undefined"
check, in which case the variable not being defined doesn't lead to an exception.The text was updated successfully, but these errors were encountered: