Skip to content

feat: update documents - #45

Open
aroldanju wants to merge 1 commit into
pooranjoyb:masterfrom
aroldanju:feature/update-document
Open

aroldanju wants to merge 1 commit into
pooranjoyb:masterfrom
aroldanju:feature/update-document

Conversation

@aroldanju

Copy link
Copy Markdown

No description provided.

@pooranjoyb pooranjoyb left a comment

Copy link
Copy Markdown
Owner

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
Copy Markdown
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
Copy Markdown
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

This branch has not been deployed

No deployments
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