Skip to content

fix: Update country-iso-search to v0.1.2 - #7994

Merged
camdecoster merged 2 commits into
mainfrom
cam/7993/update-country-iso-search
Aug 27, 2026
Merged

fix: Update country-iso-search to v0.1.2#7994
camdecoster merged 2 commits into
mainfrom
cam/7993/update-country-iso-search

Conversation

@camdecoster

@camdecoster camdecoster commented Aug 27, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Update country-iso-search to v0.1.2.

Closes #7993.

Changes

  • Update dependency

Testing

  • Be on main
  • Run npm ci
  • Save the following html into the root project folder and open it in a Chromium based browser: 
     <!doctype html>
 <script src="build/plotly.js"></script>
 <div id="gd"></div>
 <script>
 Plotly.newPlot("gd", /* JSON object */ {
     "data": [{ "y": [1, 2, 3] }],
     "layout": { "width": 600, "height": 400}
 })
 </script>
  • Note the error in the devtools console
  • Switch to this branch
  • Run npm ci
  • Reload the html file
  • Note that no error occurs and the plot renders

Notes

  • country-iso-search runs a sanitize function that previously looked for literal Unicode characters in regex patterns
  • This caused errors for pages loading plotly.js without specifying that UTF-8 encoding should be used
  • The literal characters were changed to escaped ASCII characters to address the issue

@camdecoster
camdecoster merged commit 1fe4941 into main Aug 27, 2026
84 of 85 checks passed
@camdecoster
camdecoster deleted the cam/7993/update-country-iso-search branch August 27, 2026 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KoolADE85 KoolADE85 KoolADE85 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG]: Unable to load Plotly 4 due to missing utf8

2 participants

@camdecoster @KoolADE85