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

[BUG]: Nested array display is ambiguous #385

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

[BUG]: Nested array display is ambiguous #385

liao02x opened this issue Jan 30, 2024 · 2 comments · May be fixed by #431
Assignees
Labels
approved Confirmed and ready for development. bug Something isn't working open to review Please share your insights about this issue/PR

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
@AykutSarac AykutSarac added the approved Confirmed and ready for development. label Nov 17, 2024
@AykutSarac AykutSarac pinned this issue Dec 18, 2024
@AykutSarac AykutSarac added the open to review Please share your insights about this issue/PR label Dec 18, 2024
@AykutSarac
Copy link
Owner

I appreciate the reviews for #431

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Confirmed and ready for development. bug Something isn't working open to review Please share your insights about this issue/PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants