Add UncodeJsonEncoder (fixes #42847). #87147
Conversation
ghost
added
the
community-contribution
|
Tagging subscribers to this area: @dotnet/area-system-text-encodings-web |
|
This is just the product code part, not tests; I intentionally omitted those for now to focus the discussion on the product code side first. |
|
The API shape seems consistent with what we might be looking for in #42847. Before we can entertain an implementation this would need to go through API review. TL;DR an issue with an API proposal needs to be brought in for review. Going forward I would recommend contributing to an API proposal in #42847 and we can try bringing it in for review. In the meantime, is it ok if you could convert this PR to a draft? We're tracking time-to-merge metrics for open PRs and it's unlikely this would get merged anytime soon.. |
|
@eiriktsarpalis I looked around for a button to convert to draft, but I don't see one. Feel free to click it if it shows up for you. |
|
FYI - I opened an API proposal as well (#87153). |
Also fixes dotnet#86800. Also fixes dotnet#87138 (except docs outside this repo).
davidmatson
force-pushed
the
user/davidmatson/unicodeJsonEncoder
branch
from
5dc851e
to
04fe35e
Compare
ghost
closed this
|
Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it. |
ghost
locked as resolved and limited conversation to collaborators
Also fixes #86800.
Also fixes #87138 (except docs outside this repo).