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

DeleteXXXnavigationNode does not work with "%20" in URL #864

Closed
SubPointSupport opened this issue Aug 4, 2016 · 1 comment
Closed

DeleteXXXnavigationNode does not work with "%20" in URL #864

SubPointSupport opened this issue Aug 4, 2016 · 1 comment
Assignees
@SubPointSupport
Labels
bug community-feedback
Milestone
2016-08-08

Comments

@SubPointSupport
Copy link
Contributor

SubPointSupport commented Aug 4, 2016 •
edited

The following regression tests are to be added:

  • Removing nodes with space and "%20" in the URL - such as "Shared%20Documents/AllDocuments.aspx"

M2 seems to have unknown behavior on such cases. Needs to be covered for both CSOM/SSOM.
@SubPointSupport SubPointSupport added this to the 2016-08-08 milestone Aug 4, 2016
@SubPointSupport SubPointSupport self-assigned this Aug 4, 2016
@SubPointSupport
Copy link
Contributor Author

The following tests pass for both CSOM/SSOM:

  • CanDeploy_DeleteQuickLaunchNavigationNodes_WithSpaces
  • CanDeploy_DeleteTopNavigationNodes_WithSpaces

SubPointSupport added a commit that referenced this issue Aug 6, 2016
@SubPointSupport
+Add more tests for DeleteXXXnavigationNode scenarios #864
6075f17 
+ passed CSOM/SSOM tests:
* CanDeploy_TopNavigationNode_With_Space
* CanDeploy_TopNavigationNode_With_PercentTwenty
SubPointSupport added a commit that referenced this issue Aug 6, 2016
@SubPointSupport
+ Add more tests for DeleteXXXnavigationNode scenarios #864
81bf103 
+ passed CSOM/SSOM tests:
* CanDeploy_QuickLaunchNavigationNode_With_Space
* CanDeploy_QuickLaunchNavigationNode_With_Space
SubPointSupport added a commit that referenced this issue Aug 6, 2016
@SubPointSupport
+ Add more tests for DeleteXXXnavigationNode scenarios #864
b22afe2 
+ passed CSOM/SSOM tests:
* CanDeploy_QuickLaunchNavigationNode_With_Space
* CanDeploy_QuickLaunchNavigationNode_With_Space
@SubPointSupport SubPointSupport changed the title Add more tests for DeleteXXXnavigationNode scenarios DeleteXXXnavigationNode does not work with "%20" in URL Aug 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SubPointSupport SubPointSupport

Labels
bug community-feedback
Projects
None yet
Milestone
2016-08-08
Development

No branches or pull requests
1 participant
@SubPointSupport