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

[V3.8] - Package uniforms-bridge-json-schema regex problem on safari #1088

Closed
yudao opened this issue Feb 14, 2022 · 2 comments · Fixed by #1090
Closed

[V3.8] - Package uniforms-bridge-json-schema regex problem on safari #1088

yudao opened this issue Feb 14, 2022 · 2 comments · Fixed by #1090
Assignees
Labels
Type: Bug Bug reports and their fixes
Milestone

Comments

@yudao
Copy link

yudao commented Feb 14, 2022

Affected Package: uniforms-bridge-json-schema
Affected version: 3.8
Error after run npm run dev: SyntaxError: Invalid regular expression: invalid group specifier name
Affected OS: Safari

Regex problem uniquely on Safari (This is because Safari browser does not support regular expression zero assertion in two// formats:<= , ?<!).

Chrome is OK.

I have tested all my packages and only the upgrade from uniforms-bridge-json-schema from 3.6.2 to 3.8 create this error.

@JanMP
Copy link

JanMP commented Feb 14, 2022

Same problem here. Works with 3.7.0 but breaks on 3.8.

edit: same problem but different bridge: I got it on simple-schema-2 bridge.

@wadamek65
Copy link
Contributor

Hi,

Thanks for reporting the problem. The issue was most likely introduced in: 331f717 and is probably related to this RegEx.

@wadamek65 wadamek65 added the Type: Bug Bug reports and their fixes label Feb 14, 2022
@radekmie radekmie self-assigned this Feb 15, 2022
@radekmie radekmie added this to the v3.8 milestone Feb 15, 2022
@radekmie radekmie linked a pull request Feb 15, 2022 that will close this issue
@radekmie radekmie moved this to Closed in Open Source Nov 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Bug reports and their fixes
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants