Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Move relative to absolute docs path#2949

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

MichelZ
Copy link
Contributor

Moving relative links to learn.microsoft.com, also verify msdn/docs .microsoft.com links and replacing them with learn.microsoft.com.

.xml Documentation files only

Fixes#2883

@benrr101
Copy link
Contributor

Thank you for your contribution to this! I'm glad someone was keen enough to take on this annoying task!

I'm going to have to look into this a little bit before signing off on it. According to@cheenamalhotra, we get warnings from [something] when absolute links are used. So it's an unfortunate situation where if we use relative links, the links don't work in IDEs but the doc websites are happy, and vice versa if we use absolute links.

@benrr101benrr101 added the Code Health 💊Issues/PRs that are targeted to source code quality improvements. labelNov 1, 2024
@benrr101benrr101 added this to the6.0-preview3 milestoneNov 1, 2024
@MichelZ
Copy link
ContributorAuthor

Sure, let me know where you see the error and I can also look for creative workarounds ;)

@benrr101
Copy link
Contributor

Tbh, I'm not sure exactly where this error comes in. My assumption is that it comes from our doc website generation process, but I don't know what that is. Need@cheenamalhotra's input on that

@benrr101
Copy link
Contributor

/azp run

@azure-pipelinesAzure Pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@cheenamalhotra
Copy link
Member

I'm going to have to look into this a little bit before signing off on it. According to@cheenamalhotra, we get warnings from [something] when absolute links are used. So it's an unfortunate situation where if we use relative links, the links don't work in IDEs but the doc websites are happy, and vice versa if we use absolute links.

I believe we will get warnings in .NET API reference, when the docs will be published. Something we should follow up internally, if this will be allowed with the .NET API reference docs team, because the XML files are the same that are published to both VS intellisense and .NET API reference Microsoft documentation.

Copy link
Contributor

@benrr101benrr101 left a comment

Choose a reason for hiding this comment

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

I reached out to@gewarren to get more info on this and according to her, it's perfectly acceptable to use learn.microsoft.com as the base of our URLs. Let's go ahead and take these changes.

@MichelZ thanks for doing the tedious work (I hope you used a script or something!) :D

cheenamalhotra reacted with heart emoji
Copy link
Contributor

@mdaiglemdaigle left a comment

Choose a reason for hiding this comment

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

One broken link, otherwise looks good!

Copy link
Member

@cheenamalhotracheenamalhotra left a comment

Choose a reason for hiding this comment

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

Dotnet framework links are obsolete, and don't apply to Microsoft.Data.SqlClient documentation. Use sql/connect/ado-net instead.

MichelZand others added3 commitsNovember 8, 2024 07:09
Co-authored-by: Malcolm Daigle <daiglemalcolm@gmail.com>
Co-authored-by: Cheena Malhotra <13396919+cheenamalhotra@users.noreply.github.com>
@cheenamalhotracheenamalhotra merged commit35c317b intodotnet:mainNov 13, 2024
7 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@benrr101benrr101benrr101 approved these changes

@mdaiglemdaiglemdaigle approved these changes

@cheenamalhotracheenamalhotracheenamalhotra approved these changes

Assignees
No one assigned
Labels
Code Health 💊Issues/PRs that are targeted to source code quality improvements.
Projects
None yet
Milestone
6.0-preview3
Development

Successfully merging this pull request may close these issues.

External links in documentation are relative
4 participants
@MichelZ@benrr101@cheenamalhotra@mdaigle

[8]ページ先頭

©2009-2025 Movatter.jp