Skip to content

Conversation

aroldanju
Copy link

No description provided.

Copy link
Owner

@pooranjoyb pooranjoyb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the next release we're planning to remove the nholman/json dependency and instead prepare the JSON with raw string.

You can refer to this commit id for the implementation.

Thanks for your contribution 👍🏼

std::string databaseId = "database123";
std::string collectionId = "test1234";
std::string documentId = "newDocument";
json data = {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create the payload using the raw strings


std::string url = Config::API_BASE_URL + "/databases/" + databaseId + "/collections/" + collectionId + "/documents/" + documentId;

json payloadJson = {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the nholman/json dependency

@pooranjoyb pooranjoyb added the commitment-issues basically stale PR label Apr 8, 2025
@pooranjoyb pooranjoyb moved this to Todo in C++ SDK Appwrite Jul 3, 2025
@pooranjoyb pooranjoyb moved this from Todo to In Progress in C++ SDK Appwrite Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commitment-issues basically stale PR

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants