Skip to content

Conversation

@demiracl
Copy link
Contributor

@demiracl demiracl commented Jul 7, 2021

Updated Policy urls to point to Docs v7

@demiracl demiracl requested a review from a team as a code owner July 7, 2021 08:57
@darcyclarke darcyclarke added the Release 7.x work is associated with a specific npm 7 release label Jul 12, 2021
@wraithgar
Copy link
Member

Those urls currently 404

@wraithgar wraithgar marked this pull request as draft July 12, 2021 16:52
@wraithgar
Copy link
Member

Converting to draft so we can visually see that this isn't ready till those urls exist.

@demiracl
Copy link
Contributor Author

The urls in this PR depend on the documentation PR - npm/documentation#20 to be merged, so they resolve

@demiracl demiracl marked this pull request as ready for review July 15, 2021 08:47
Copy link
Member

@wraithgar wraithgar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spot checked the urls they seem to be working now.

@wraithgar wraithgar changed the base branch from latest to release-next July 22, 2021 13:42
@wraithgar wraithgar force-pushed the update-policy-urls-v7 branch from 57fccea to 51e9d1b Compare July 22, 2021 13:54
PR-URL: #3523
Credit: @DemiraDimitrova
Close: #3523
Reviewed-by: @wraithgar
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release 7.x work is associated with a specific npm 7 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants