Skip to content

Add complex Json Example #738

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

omkenge
Copy link
Contributor

@omkenge omkenge commented Apr 24, 2025

No description provided.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions github-actions bot added status:awaiting review PR awaiting review from a maintainer component:examples Issues/PR referencing examples folder labels Apr 24, 2025
@omkenge
Copy link
Contributor Author

omkenge commented Apr 29, 2025

cc @Giom-V

@Giom-V
Copy link
Collaborator

Giom-V commented May 6, 2025

Thanks @omkenge for the submission. I left some minor comments.

@andycandy
Copy link
Collaborator

Do you still want to push this example @omkenge?

@omkenge
Copy link
Contributor Author

omkenge commented Jun 5, 2025

@andycandy Thanks for reminder I will complete this PR on weekend.

Copy link
Contributor Author

omkenge commented Jun 7, 2025

@andycandy Done Again Thanks for reminder.

@omkenge omkenge requested a review from Giom-V June 7, 2025 09:42
@@ -46,7 +46,7 @@
"id": "bxGr_x3MRA0z"
Copy link
Collaborator

@andycandy andycandy Jun 18, 2025

Choose a reason for hiding this comment

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

I think it’d be great if you could wrap the long string into multiple lines for readability.


Reply via ReviewNB

@andycandy
Copy link
Collaborator

Also make sure to resolve the merge conflicts, everything else LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:examples Issues/PR referencing examples folder status:awaiting review PR awaiting review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants