Skip to content

Add tojson-document, tojson-dict, and tojson-array for CBL-C #539

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

Merged
merged 2 commits into from
Sep 16, 2021

Conversation

pasin
Copy link
Contributor

@pasin pasin commented Sep 15, 2021

  • Added tojson-document, tojson-dict, tojson-array for CBL-C
  • The code snippet in tojson-document and tojson-array are the proposed simplified version (DOC-9157 and DOC-9147)
  • Not providing the code snippet for tojson-blob as it’s not the main usecase and could lead to confusion (DOC-9149). I’m proposing the remove the entire section for this.
  • Fixed build issue

* Added tojson-document, tojson-dict, tojson-array for CBL-C
* The code snippet in tojson-document and tojson-array are the proposed simplified version (DOC-9157 and DOC-9147)
* Not providing the code snippet for tojson-blob as it’s not the main usecase and could lead to confusion (DOC-9149). I’m proposing the remove the entire section for this.

# Conflicts:
#	modules/c/examples/code_snippets/main.c
@pasin pasin requested a review from ibsoln September 15, 2021 21:30
To avoid confusing with FLDoct, update to use FLData_ConvertJSON and FLDoc_FromJSON.
Copy link
Contributor

@ibsoln ibsoln left a comment

Choose a reason for hiding this comment

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

Happy with this @pasin ; thank you.
I'll address the Jira tickets asap.

@ibsoln ibsoln merged commit 9c5a664 into release/3.0 Sep 16, 2021
@ibsoln ibsoln deleted the feature/DOC-8766 branch September 16, 2021 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants