Skip to content

Conversation

reakaleek
Copy link
Member

@reakaleek reakaleek commented Jan 7, 2025

Closes #140

Changes

Add a sentence on how to fix the warning, explaining where to add the base domain.

Warning: External URI 'https://docs.elastic.dev/migration/site-level/content#asciidoctor-conf-yml' is not allowed. Add 'elastic.dev' to the 'external_hosts' list in /Users/jcal/Projects/docs-builder/docs/source/docset.yml to allow links to this domain.
   ┌─[/Users/jcal/Projects/docs-builder/docs/source/testing/index.md]
   │
   │
   └─

Notes

The warning is missing the actual markdown line where the error is happening. (See example from #140)
I saw the same behavior with @Mpdreamz when he demoed the app to me.

@reakaleek reakaleek requested a review from Mpdreamz January 7, 2025 11:24
@reakaleek reakaleek self-assigned this Jan 7, 2025
@reakaleek reakaleek marked this pull request as ready for review January 7, 2025 11:32
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

LGTM!

@reakaleek reakaleek merged commit cc8fdf5 into main Jan 7, 2025
4 checks passed
@reakaleek reakaleek deleted the feature/better-warning-message-for-external-links branch January 7, 2025 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't link to docs.elastic.dev: Warning: External URI

2 participants