Skip to content

Conversation

@B4nan
Copy link
Member

@B4nan B4nan commented Nov 13, 2024

No description provided.

@B4nan B4nan requested a review from TC-MO as a code owner November 13, 2024 13:09
@github-actions github-actions bot added this to the 102nd sprint - Tooling team milestone Nov 13, 2024
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Nov 13, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

Pull request is neither linked to an issue or epic nor labeled as adhoc!

@B4nan B4nan added the adhoc Ad-hoc unplanned task added during the sprint. label Nov 13, 2024
@B4nan B4nan requested review from janbuchar and removed request for TC-MO November 13, 2024 13:28
Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

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

Should there be a bigger audit to double check all API links?

@B4nan
Copy link
Member Author

B4nan commented Nov 13, 2024

Should there be a bigger audit to double check all API links?

Most likely, this is fixing the one I know about, but more importantly, it fixes the anchor position. Then there is a separate issue you were facing with the scroll happening too fast and you end up at the bottom anyway, but first we need to have correct anchor position and correct link.

FYI I went through all dataset links in the docs from this repo and this is the only one that was in wrong format without the middle part.

@B4nan B4nan merged commit c73b618 into master Nov 13, 2024
7 checks passed
@B4nan B4nan deleted the fix-api-links branch November 13, 2024 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants