JsonApiDotNetCore icon indicating copy to clipboard operation
JsonApiDotNetCore copied to clipboard

Fix broken doc links

Open leandroaraujosouza opened this issue 3 years ago • 3 comments

This PR fixes broken links in the docs/usage/extensibility/resource-definitions.md file

Closes #{ISSUE_NUMBER}

QUALITY CHECKLIST

  • [ ] Changes implemented in code
  • [X] Complies with our contributing guidelines
  • [ ] Adapted tests
  • [X] Documentation updated

leandroaraujosouza avatar Jul 21 '22 01:07 leandroaraujosouza

Codecov Report

Merging #1171 (bc18441) into master (0185311) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1171   +/-   ##
=======================================
  Coverage   92.58%   92.58%           
=======================================
  Files         241      241           
  Lines        7700     7700           
=======================================
  Hits         7129     7129           
  Misses        571      571           

codecov[bot] avatar Jul 21 '22 02:07 codecov[bot]

Hi @leandroaraujosouza, thanks for taking the time to create a PR. Though I'm confused: where do you see these links are broken? They seem to render properly at https://www.jsonapi.net/usage/extensibility/resource-definitions.html. The ~ paths are resolved by docfx, which we use to build the documentation website. We use these links so that docfx validates them and reports build warnings when they break.

bkoelman avatar Jul 21 '22 18:07 bkoelman

Hi @leandroaraujosouza, thanks for taking the time to create a PR. Though I'm confused: where do you see these links are broken? They seem to render properly at https://www.jsonapi.net/usage/extensibility/resource-definitions.html. The ~ paths are resolved by docfx, which we use to build the documentation website. We use these links so that docfx validates them and reports build warnings when they break.

Hi @bkoelman I see them broken when browsing through the repo -> https://github.com/json-api-dotnet/JsonApiDotNetCore/blob/master/docs/usage/extensibility/resource-definitions.md

image

I'll be reading more about docfx to understand better so I can propose a solution that works in both cases.

leandroaraujosouza avatar Jul 21 '22 23:07 leandroaraujosouza

Hi @leandroaraujosouza, are you still working on this?

bkoelman avatar Aug 21 '22 14:08 bkoelman

Closing due to inactivity.

bkoelman avatar Sep 03 '22 16:09 bkoelman