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.

Sign up for GitHub

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

Jump to bottom

[BUG]: Nested array display is ambiguous #385

Open
liao02x opened this issue Jan 30, 2024 · 1 comment · May be fixed by #431
Open

[BUG]: Nested array display is ambiguous #385

liao02x opened this issue Jan 30, 2024 · 1 comment · May be fixed by #431
Assignees
@AykutSarac
Labels
approved Confirmed and ready for development. bug Something isn't working

Comments

@liao02x
Copy link

liao02x commented Jan 30, 2024

Issue description

Different array structure (one nested array, one simple array), same display

Media & Screenshots

Screenshot from 2024-01-29 23-56-14
Screenshot from 2024-01-29 23-54-53

Operating system

  • OS: [e.g. iOS]:

  • Browser [e.g. chrome, safari]:

  • Any other details...

Priority this issue should have

Low (slightly annoying)
@liao02x liao02x added the bug Something isn't working label Jan 30, 2024
@AykutSarac
Copy link
Owner

New algorithm and new roadmap requires too much time and effort, thus new visualization along with bug fixes are released under the premium plan that is part of the new editor.

If somebody is willing to fix this; there should be a root node created for the array, and it shouldn't break the JSON path finding algorithm as well. Open to PRs to fix this issue.

@acpaquette acpaquette linked a pull request Nov 14, 2024 that will close this issue
Open
@AykutSarac AykutSarac added the approved Confirmed and ready for development. label Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AykutSarac AykutSarac

Labels
approved Confirmed and ready for development. bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update Json Graph Traversal Algorithm acpaquette/jsoncrack.com
2 participants
@liao02x @AykutSarac