-
Notifications
You must be signed in to change notification settings - Fork 486
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
docs: fix 'Flow mode' link #5412
Conversation
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.
I'm preemptively approving, but leaving it to @clayton-cornell for the final decision.
I'm not sure why the link is not working here; does it have to do with it being inside of the admonition block?
@d4nyll Thanks for spotting this one. It slipped through on me. The link is failing to render correctly because it's inside a note block, and the link definition is set in a The simplest fix here is to simply remove the link from the Note. We have this link elsewhere within the same topic so we aren't loosing anything. Also... we can't update directly within this So do you want to roll back this change and apply it to the |
Hmmm... looking deeper into this one, and it looks like the fix is a bit complicated because it happened right around when we templated this |
Fixed in:
Closing this PR. Thanks for finding this mistake @d4nyll The fix took a little fiddling and two separate PRs to get it all right. :-) |
PR Description
Fix link to Flow Mode in docs: