Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove or internalize JSON.NET dependency in Cosmos Provider #14570

Open
ajcvickers opened this issue Jan 31, 2019 · 5 comments
Open

Remove or internalize JSON.NET dependency in Cosmos Provider #14570

ajcvickers opened this issue Jan 31, 2019 · 5 comments
Assignees
Labels
area-cosmos breaking-change consider-for-current-release punted-for-6.0 punted-for-7.0 Originally planned for the EF Core 7.0 (EF7) release, but moved out due to resource constraints. type-enhancement
Milestone

Comments

@ajcvickers
Copy link
Member

If this is not possible, then we should:

  • Give feedback to the SDK team
  • Raise this issue with respect to 3.0 shipping and come up with a plan
@AndriySvyryd
Copy link
Member

AndriySvyryd commented Jan 31, 2019

We need to wait until JsonConverter(https://github.com/dotnet/corefx/issues/34372) is available to be able to implement CosmosJsonSerializer
Also see CosmosSystemTextJsonSerializer

@ajcvickers ajcvickers added this to the 3.0.0 milestone Feb 1, 2019
@ajcvickers
Copy link
Member Author

See also: Azure/azure-cosmos-dotnet-v3#37

@ajcvickers ajcvickers removed this from the 3.0.0 milestone Feb 1, 2019
@ajcvickers
Copy link
Member Author

Confirmed that we don't need to do this for 3.0.

@AndriySvyryd
Copy link
Member

AndriySvyryd commented Jun 17, 2020

Now this is blocked on writable DOM support: dotnet/runtime#29690

@AndriySvyryd
Copy link
Member

Note to implementor: Add byte[] converter for backwards compatibility dotnet/runtime#30456

@ajcvickers ajcvickers modified the milestones: 6.0.0, Backlog Jul 30, 2021
@ajcvickers ajcvickers modified the milestones: Backlog, 7.0.0 Oct 27, 2021
@ajcvickers ajcvickers added punted-for-7.0 Originally planned for the EF Core 7.0 (EF7) release, but moved out due to resource constraints. and removed propose-punt labels Jul 7, 2022
@ajcvickers ajcvickers modified the milestones: 7.0.0, Backlog Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-cosmos breaking-change consider-for-current-release punted-for-6.0 punted-for-7.0 Originally planned for the EF Core 7.0 (EF7) release, but moved out due to resource constraints. type-enhancement
Projects
None yet
Development

No branches or pull requests

3 participants