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

Bugfix: missing links for invariant documents with hostnames #17808

Merged

Conversation

madsrasmussen
Copy link
Contributor

Fixes: #17793

This PR ensures that we always render all the document URLs we get from the server. I have refactored the code so we still include a URL status for variant documents even though the server doesn't return URLs for all variants.

What to test:
Please at least test the following cases:

  • Invariant Document with no hostnames
  • Invartiant Document with 1 hostname
  • Invariant Document with multiple hostnames
  • The same for variant documents
  • New Documents
  • Unpublished Documents
  • Published Documents

Copy link
Contributor

@iOvergaard iOvergaard left a comment

Choose a reason for hiding this comment

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

LGTM

@iOvergaard iOvergaard merged commit b0b4571 into v15/dev Dec 18, 2024
28 of 29 checks passed
@iOvergaard iOvergaard deleted the v15/bugfix/missing-links-for-invariant-with-hostnames branch December 18, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants