ampproject / ampproject/amp-toolbox
cache-url: Remove TextEncoder
Open
- 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
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