ampproject / ampproject/amp-toolbox

cache-url: Remove TextEncoder

Open
#135 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
HTML
Stars
459
Forks
242
PR merge metrics
No merged PRs in 30d

Description

see #132

TextEncoder is not supported on some browsers, thus it would be good to remove this and use something else.

cc @sebastianbenz @kristoferbaxter

Contributor guide

Open the contributing guide

Research direction

Start with issue #132 and locate the cache-url implementation that uses TextEncoder. Review the browser compatibility concern and identify the existing tests or entry point for cache-url; done means the unsupported API is no longer required and the affected behavior remains covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
html
Domain
web-dev
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.