Skip to content

Writing JSON data Tutorial - C# Language Update#190

Merged
omckeon merged 14 commits intothoth-tech:tutorial-guide-improvementsfrom
Srainyyyy:Writing-JSON-data-Tutorial---C#-Language-Update
Oct 1, 2024
Merged

Writing JSON data Tutorial - C# Language Update#190
omckeon merged 14 commits intothoth-tech:tutorial-guide-improvementsfrom
Srainyyyy:Writing-JSON-data-Tutorial---C#-Language-Update

Conversation

@Srainyyyy
Copy link

I added c# OOP statements to the sample code and made sure that each code block was added. The code running screenshot is as follows:
Writing JSON Data to a File
0d8125c518b0e584f1b55dd47ee89d9e
Modifying Existing JSON Data
d8b0ec6f8b3dac3c1a706f4e5f7b09c5

@netlify
Copy link

netlify bot commented Sep 20, 2024

Deploy Preview for splashkit-io ready!

Name Link
🔨 Latest commit 4098ac4
🔍 Latest deploy log https://app.netlify.com/sites/splashkit-io/deploys/66fb9b0bb34cec0008429d9b
😎 Deploy Preview https://deploy-preview-190--splashkit-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@kay-kaushik kay-kaushik left a comment

Choose a reason for hiding this comment

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

image

As you can see in the code snippets , the new game data argument has been written in snake case and camel case. For better readability, I would suggest following one naming convention.

@kay-kaushik
Copy link

Everyting else looks good.

Copy link

@Jinx-means-jinx Jinx-means-jinx left a comment

Choose a reason for hiding this comment

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

Testing the code for fill_triangle_on_bitmap
C++:
Test-Fill-trangle-on-batman
C#:
Test-Fill-trangle-on-batman-c#
Code working with no Error.
Overall Look great!

@omckeon omckeon changed the base branch from main to tutorial-guide-improvements October 1, 2024 06:38
@omckeon omckeon merged commit 6b8688c into thoth-tech:tutorial-guide-improvements Oct 1, 2024
@netlify
Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for tutorials-splashkit-io ready!

Name Link
🔨 Latest commit 4098ac4
🔍 Latest deploy log https://app.netlify.com/sites/tutorials-splashkit-io/deploys/66fb9b0bdf757c0008e29e00
😎 Deploy Preview https://deploy-preview-190--tutorials-splashkit-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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.

5 participants