-
Notifications
You must be signed in to change notification settings - Fork 969
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
Fix heading and links for loaded_at_field
#5207
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
(Required) | ||
</VersionBlock> | ||
A column name (or expression) that returns a timestamp indicating freshness. | ||
|
||
<br/><br/>A column name (or expression) that returns a timestamp indicating freshness. |
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.
Adding the <br/><br/>
tags is a little ugly, but it's the only thing I found that would work to preserve the current formatting.
I tried several things to no avail.
If we don't like them, I'd suggest that we merge this PR as-is and create a follow-up to address them.
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.
lgtm and i believe this may be related to a potential bug that i reported to the front end team here.
i had another pr where i did something similar to what you did here in order to solve this until it's permanently fixed.
Preview 1.6
Preview 1.7
What are you changing in this pull request and why?
The changes in this PR fix the following two issues:
There are two links on the sidebar for
loaded_at_field
:Another issue is that links to the
#loaded_at_field
anchor tag won't always go where you intend, depending on the version the user has selected. I believe this has the same root cause as the first issue.🎩
After
1.6 before
1.6 after
1.7 before
1.7 after
Checklist