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

feat(summary): add strategy which categorizes current directory only #1264

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

nuhakala
Copy link

@nuhakala nuhakala commented Jan 6, 2024

Why

I like to structure my notes in hierarchial manner inside folders. However, the workspace summary creates a summary of the whole workspace, even though I would like to create a summary of only current directory, creating links to subfolder index files. And that is what these modifications do.

What

I added new strategy current_dir which creates links to subfolder index.norg files and categorizes the regular files in the directory as before. If subdirectory does not contain index.norg file, it is not created and the link is not added.

Open points / problems

  • What I did, I just copied the default strategy and modified the beginning of it a little. However, this leads the module to have quite a lot of duplicated code.
  • I also had to read the files of the current directory again, which is inefficient.

P.S. This is my first pull request to open source project ever. I hope the changes are not too bad.

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.

None yet

1 participant