Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Bug: doc link for Notifications (Beta) samples in GE return error 404 #353

Closed
Shjokie opened this issue Sep 12, 2019 · 0 comments · Fixed by #355 or #357
Closed

Bug: doc link for Notifications (Beta) samples in GE return error 404 #353

Shjokie opened this issue Sep 12, 2019 · 0 comments · Fixed by #355 or #357
Assignees
Labels
Milestone

Comments

@Shjokie
Copy link
Contributor

Shjokie commented Sep 12, 2019

Expected behavior

When you click the link to get the docs, you should get the page in the screenshot below.
image

Link should be: https://docs.microsoft.com/en-us/graph/api/resources/projectrome-notification?view=graph-rest-beta

Actual behavior

The link resolves to this: https://docs.microsoft.com/en-us/graph/api/projectrome-notification-post?view=graph-rest-beta which bring the error in the screenshot below:
image

Steps to reproduce the behavior

  1. Go to https://developer.microsoft.com/en-us/graph/graph-explorer#
  2. Click on "More samples" under the Sample Queries
  3. Activate the Notifications (beta) sample
  4. Close the More samples side bar
  5. Click on the link to documentation that's beside the "create a raw notification" and "Create a visual notification" sample queries
  6. It will redirect to a page with error 404

HTTP traffic capture

@Shjokie Shjokie added the bug label Sep 12, 2019
@Shjokie Shjokie added this to the 1.8.x milestone Sep 12, 2019
@Shjokie Shjokie added this to To do in Maintenance Work via automation Sep 12, 2019
@andrueastman andrueastman assigned andrueastman and unassigned jobala Sep 16, 2019
@andrueastman andrueastman moved this from To do to In progress in Maintenance Work Sep 16, 2019
@andrueastman andrueastman moved this from In progress to To Merge in Maintenance Work Sep 16, 2019
Maintenance Work automation moved this from To Merge to Done Sep 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.