Closed josephjclark closed 4 months ago
The previous fix to UTF8, which introduced loadAnyAscii, didn't quite work.
This PR basically just wraps that function in a function, which ensures that execute properly waits for the module to load.
execute
This now works against a local worker, so I'm more confidence it'll work in production.
Also note that to support this fix I've improved the unit tests on toUTF8. They now use execute, which forces the dynamic module load of any-ascii
toUTF8
any-ascii
The previous fix to UTF8, which introduced loadAnyAscii, didn't quite work.
This PR basically just wraps that function in a function, which ensures that
execute
properly waits for the module to load.This now works against a local worker, so I'm more confidence it'll work in production.
Also note that to support this fix I've improved the unit tests on
toUTF8
. They now useexecute
, which forces the dynamic module load ofany-ascii