Skip to content

Fix: #3621 - AddNavigationNode - Search in sub nodes for id#3625

Merged
gautamdsheth merged 2 commits intopnp:devfrom
JuergenRB:feature/GH-3621
Jan 9, 2024
Merged

Fix: #3621 - AddNavigationNode - Search in sub nodes for id#3625
gautamdsheth merged 2 commits intopnp:devfrom
JuergenRB:feature/GH-3621

Conversation

@JuergenRB
Copy link
Contributor

@JuergenRB JuergenRB commented Dec 7, 2023

Type

  • Bug Fix

Related Issues?

Issue #3621

What is in this Pull Request ?

Within the AddNavigationNode command:

NavigationNode property Nodes now also contains NavigationNodes instead of objects.
To find a node by ID, not only the first list but also all child nodes are searched.

@gautamdsheth
Copy link
Collaborator

@JuergenRB - is this ready ? Have you tested the code ?

@JuergenRB JuergenRB marked this pull request as ready for review December 30, 2023 22:59
@JuergenRB
Copy link
Contributor Author

@gautamdsheth - Yes, the change is finished and tested

@gautamdsheth gautamdsheth merged commit 88985d0 into pnp:dev Jan 9, 2024
@gautamdsheth
Copy link
Collaborator

Thanks @JuergenRB , good catch, much appreciated the fix for the issue !!

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