Skip to content
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

implemented table of contents for each page #8

Merged
merged 9 commits into from
Apr 3, 2022

Conversation

dsumer
Copy link
Collaborator

@dsumer dsumer commented Mar 29, 2022

No description provided.

src/utils/toc.ts Outdated Show resolved Hide resolved
src/utils/toc.ts Outdated Show resolved Hide resolved
src/utils/toc.ts Outdated Show resolved Hide resolved
src/utils/toc.ts Outdated Show resolved Hide resolved
src/utils/toc.ts Outdated Show resolved Hide resolved
src/utils/markdown.ts Outdated Show resolved Hide resolved
@dsumer
Copy link
Collaborator Author

dsumer commented Mar 29, 2022

I've added a test but sadly it fails because the "github-slugger" module is undefined when the test is executed .. needed to mock it in order to create the test

do you maybe know why this is the case? did you have similar problems that node_module packages are undefined in tests?

@frouo
Copy link
Owner

frouo commented Mar 29, 2022

That's absolutely awesome @dsumer, thank you 🎉 + tests wow 👏

Regarding your last comment, yes, we still have packaging issue, sorry about that. @edvincandon is working on it (cf. #10) and we will fix that in priority.

@frouo
Copy link
Owner

frouo commented Apr 3, 2022

☝🏻 rebased the branch on top of master

@frouo frouo merged commit 6b18ad4 into frouo:master Apr 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants