azure-docs icon indicating copy to clipboard operation
azure-docs copied to clipboard

Refer to mysql2 instead of mysql for Node.js

Open Zenulous opened this issue 2 years ago • 5 comments

The Mysql npm package has not been updated in years. Consider referring to https://www.npmjs.com/package/mysql2 instead.

Source: https://learn.microsoft.com/en-us/azure/mysql/single-server/connect-nodejs


Document Details

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

Zenulous avatar Dec 13 '22 21:12 Zenulous

Seems like GitHub mobile got rid of the initial document details 🤔

Zenulous avatar Dec 13 '22 22:12 Zenulous

@Zenulous

Thanks for your feedback! We will investigate and update as appropriate.

Hi @savjani,

Could you please look into this document update request for package update at the link: https://www.npmjs.com/package/mysql

image

shaktisingh-msft avatar Dec 14 '22 05:12 shaktisingh-msft

Adding @shreyaaithal to validate and update the doc

savjani avatar Dec 14 '22 06:12 savjani

FYI it's not like mysql is broken but after playing around with it this seems heavily outdated. For example mysql does not support promises and mysql2 does 😄

Zenulous avatar Dec 14 '22 08:12 Zenulous

Thank you @Zenulous for the suggestion! The node.js quickstarts have been updated accordingly.

shreyaaithal avatar Jun 21 '23 11:06 shreyaaithal

#please-close

shreyaaithal avatar Jun 21 '23 11:06 shreyaaithal