Skip to content
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

updated policy urls - v7 #3523

Merged
merged 1 commit into from Jul 22, 2021
Merged

updated policy urls - v7 #3523

merged 1 commit into from Jul 22, 2021

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
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.

None yet

3 participants