Azure-docs: Old library and broken link

[dhulme]

This page refers to azure-sb package, but there's a newer @azure/service-bus package. Can/should we use this for notifications?

Also the link to the iOS notification payload documentation does not work.

---

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

• ID: 6b4435f8-c50c-57a3-c98c-c089b280afec
• Version Independent ID: 545a9fa0-0f61-4a67-4de0-7356fe399259
• Content: Sending push notifications with Azure Notification Hubs and Node.js
• Content Source: articles/notification-hubs/notification-hubs-nodejs-push-notification-tutorial.md
• Service: notification-hubs
• GitHub Login: @sethmanheim
• Microsoft Alias: sethm

[RyanHill-MSFT]

Thanks for the feedback @dhulme! We are currently investigating and will update you shortly.

[dhulme]

Related to https://github.com/Azure/azure-sdk-for-js/issues/3729#issuecomment-619079211

[dhulme]

There's also @azure/arm-notificationhubs mentioned on https://docs.microsoft.com/en-us/javascript/api/overview/azure/notification-hubs?view=azure-node-latest

[RyanHill-MSFT]

Hi @dhulme, Azure/azure-sdk-for-node has been deprecated and replaced with Azure/azure-sdk-for-js. I also noticed your comment as well so I would advise still utilizing azure-sb until the team has integrated notification hub in the js sdk.

With regards to the missing iOS link, I've assigned this issue to the doc author due to Apple updating thier docs.

[sethmanheim]

Thank you for catching that @dhulme -- I have updated this sentence in the iOS section with a link that seems more appropriate: https://developer.apple.com/documentation/usernotifications.

[dhulme]

Thanks, looks good

[cgat]

@RyanHill-MSFT This was very confusing for me as well. I spent a fair bit of time trying to figure out how to use the new js sdk to achieve to send push notifications. The tutorial should clearly state that this is not currently supported in the new sdk.

[RyanHill-MSFT]

@RyanHill-MSFT This was very confusing for me as well. I spent a fair bit of time trying to figure out how to use the new js sdk to achieve to send push notifications. The tutorial should clearly state that this is not currently supported in the new sdk.

/cc @sethmanheim