-
Notifications
You must be signed in to change notification settings - Fork 192
Redirect rule, when the header Accept: text/markdown is set, than red… #1476
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
Conversation
…irect to the .md page
squakez
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It may not work with fragments, but I guess it is not a concern for LLMs. Or is it?
|
@squakez interesting, can you provide an example? |
|
@davsclaus @squakez @oscerd the issue with the CI might be related to https://github.com/apache/camel/blob/45d0150c311355b669e6f75f28cc96e4f09e85ca/docs/user-manual/modules/ROOT/pages/jbang-commands/examples/run.adoc?plain=1#L1 it starts with two level heading |
|
For the moment there is only the run example. So we could simply change that on the core repository and regen. I'm completing one thing and then I'll switch to that, sorry for the inconveinience |
|
No problem, I can take care of that, thanks |
|
For the moment there is just one example as "example" for camel-run |
|
apache/camel#20857 Once merged, what needs to be done so that the camel-website repo is updated? do I just need to rerun the gh action? |
|
🚀 Preview is available at https://pr-1476--camel.netlify.app |
|
Works as expected: |
…irect to the .md page
Add Accept:
text/markdowncontent negotiation for LLM access.Requests with
Accept: text/markdownheader are redirected (303) to the corresponding .md file.Example:
For reference: