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

Several JSON addition #203

Merged
merged 1 commit into from
Aug 20, 2021
Merged

Several JSON addition #203

merged 1 commit into from
Aug 20, 2021

Conversation

The-EDev
Copy link
Member

@The-EDev The-EDev commented Aug 20, 2021

  • added json list in a similar fashion to json object
  • renamed object_type to object
  • updated README.md, index.html, and json.md to include objects and lists
  • updated examples to be slightly cleaner and include lists
  • replaced instances of json object (std::map and such) with the short version (object)
  • accurate floating point number dumping (6.0 instead of 6) while taking 1/30th of the time (29 microseconds saved)
  • added json list testing
  • snuck in utf-8 middleware warning fix
  • snuck in twitter card style for https://crowcpp.org site (makes social media cards look way better with a large image)

added json list in a similar fashion to json object
renamed object_type to object
updated readme, index.html, and json.md to include objects and lists
updated examples to be slightly cleaner and include lists
replaced instances of json object (std::map and such) with the short version (object)
accurate floating point number dumping (`6.0` instead of `6`) while taking 1/30th of the time (29 microseconds saved)
added json list testing
snuck in utf-8 middleware warning fix
snuck in twitter card style for crowcpp.org site (makes social media cards look way better with a large image)
@The-EDev The-EDev requested a review from mrozigor August 20, 2021 00:58
@The-EDev
Copy link
Member Author

The-EDev commented Aug 20, 2021

Old card on Discord
image

New card on Discord
image

These cards appear under the message whenever the URL is posted

@The-EDev The-EDev changed the title several changes: Several JSON addition Aug 20, 2021
@The-EDev The-EDev merged commit a3ef00f into master Aug 20, 2021
@The-EDev The-EDev deleted the json_additions branch August 20, 2021 19:48
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