Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Update utils.ts #896

Merged
merged 1 commit into from
Dec 10, 2020
Merged

Conversation

SKlimov
Copy link
Contributor

@SKlimov SKlimov commented Dec 7, 2020

Issue #, if available:

Description of changes:
Changed encode function for saving to file.
encodeURI doesn't work very well in current versions of Chrome. This function won't escape symbols like "#".
So, all requested data that should be saved in a file will be truncated by the first # symbol in Chrome browser, in Safari this feature works fine.

@codecov
Copy link

codecov bot commented Dec 7, 2020

Codecov Report

Merging #896 (26900ac) into develop (a00a73d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop     #896   +/-   ##
==========================================
  Coverage      99.85%   99.85%           
  Complexity      2146     2146           
==========================================
  Files            216      216           
  Lines           4833     4833           
  Branches         318      318           
==========================================
  Hits            4826     4826           
  Misses             5        5           
  Partials           2        2           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a00a73d...26900ac. Read the comment docs.

Copy link
Member

@chloe-zh chloe-zh left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the fix!

@chloe-zh chloe-zh merged commit 2a932d8 into opendistro-for-elasticsearch:develop Dec 10, 2020
chloe-zh pushed a commit to chloe-zh/sql that referenced this pull request Dec 15, 2020
chloe-zh pushed a commit to chloe-zh/sql that referenced this pull request Dec 15, 2020
chloe-zh pushed a commit to chloe-zh/sql that referenced this pull request Dec 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants