-
Notifications
You must be signed in to change notification settings - Fork 325
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.
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
Restructures v10 upgrade guidance #6457
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Ready to merge 🚀 |
@milanholemans I rechecked the release blog post and we use the following links to upgrade guidance none of those were changed. So it seems we are go to go 🚀 |
All of the links you mentioned were changed, no? |
Hmmm, yes I think your right 🤦♂️ I must have checked it on wrong branch locally. |
@milanholemans I prepared the PR with required changes in the release blog post pnp/blog#1966 |
Merged manually. Awesome work 👏. You Rock 🤩 |
Closes #6363
Let's not forget to update the release blog since this change will break some of its link anchors.