Skip to content

fix: removing toc to fix the docs html#341

Merged
ST-DDT merged 3 commits intomainfrom
fix/remove-malformed-toc
Jan 30, 2022
Merged

fix: removing toc to fix the docs html#341
ST-DDT merged 3 commits intomainfrom
fix/remove-malformed-toc

Conversation

@JessicaSachs
Copy link
Contributor

@JessicaSachs JessicaSachs commented Jan 29, 2022

This removes the TOC from the codegen. It's okay. We'll improve the Docs UX shortly.

Should solve #327

@JessicaSachs JessicaSachs requested a review from a team as a code owner January 29, 2022 04:04
@JessicaSachs JessicaSachs force-pushed the fix/remove-malformed-toc branch from 584b281 to 9773f7d Compare January 29, 2022 04:05
@ejcheng ejcheng added the c: bug Something isn't working label Jan 29, 2022
ST-DDT
ST-DDT previously approved these changes Jan 29, 2022
Copy link
Member

@ST-DDT ST-DDT left a comment

Choose a reason for hiding this comment

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

Strange cause, but works.

@ST-DDT ST-DDT requested a review from a team January 29, 2022 10:54
Copy link
Member

@Shinigami92 Shinigami92 left a comment

Choose a reason for hiding this comment

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

I'm okay for now with this change/removal of toc, because in the sidebar you can still the what functions are there.

@Shinigami92 Shinigami92 added c: docs Improvements or additions to documentation c: test labels Jan 29, 2022
@Shinigami92 Shinigami92 requested a review from ST-DDT January 30, 2022 10:09
@ST-DDT ST-DDT merged commit c932942 into main Jan 30, 2022
@ST-DDT ST-DDT deleted the fix/remove-malformed-toc branch January 30, 2022 10:16
bmenant pushed a commit to bmenant/faker that referenced this pull request Mar 11, 2022
Co-authored-by: Shinigami <chrissi92@hotmail.de>
demipel8 pushed a commit to demipel8/faker that referenced this pull request Mar 11, 2022
Co-authored-by: Shinigami <chrissi92@hotmail.de>
demipel8 pushed a commit to demipel8/faker that referenced this pull request Mar 11, 2022
Co-authored-by: Shinigami <chrissi92@hotmail.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: bug Something isn't working c: docs Improvements or additions to documentation c: test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The last returns and example section on each api docs page vanishes after reload

5 participants