Skip to content

Add --flat option for database syncs#65

Closed
oliverkaiser wants to merge 4 commits intoMyastr0:masterfrom
oliverkaiser:feature/add-flat-parameter-for-db-sync
Closed

Add --flat option for database syncs#65
oliverkaiser wants to merge 4 commits intoMyastr0:masterfrom
oliverkaiser:feature/add-flat-parameter-for-db-sync

Conversation

@oliverkaiser
Copy link

@oliverkaiser oliverkaiser commented Dec 1, 2025

Added --flat parameter for database syncs, preventing nesting of pages. This is useful if you have a larger set of MD files in a folder but want to have all of them as single entries in your database instead of nested child pages. So you can filter through all the documents e.g. via their properties.


All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  1. Does your submission pass tests?
  2. Have you lint your code locally before submission?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully run tests with your changes locally?

@oliverkaiser oliverkaiser requested a review from Myastr0 as a code owner December 1, 2025 16:16
@oliverkaiser
Copy link
Author

  • added --flat option for database syncs, avoiding nested pages.

@Myastr0
Copy link
Owner

Myastr0 commented Dec 1, 2025

First thank you for your contribution on mk-notes project ! 🙌

Please can you provide a pull request description using the PR Description template file and explain a bit more what your changes are doing on mk-notes behavior 🙇‍♂️

Copy link
Owner

@Myastr0 Myastr0 left a comment

Choose a reason for hiding this comment

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

I left you few comments on your PR, can you also add a new entry in the /docs sub projet to explain the new --flat behavior 🙇‍♂️

@oliverkaiser oliverkaiser changed the title add missed files Add --flat option for database syncs Dec 1, 2025
@oliverkaiser
Copy link
Author

oliverkaiser commented Dec 3, 2025

@Myastr0 done :) Sorry for some Ai mess, I am not really an engineer anymore and now AI helps me a lot to do some smaller things like these where I have gaps. Tried to follow your structures as best as possible.

@oliverkaiser oliverkaiser requested a review from Myastr0 December 10, 2025 07:02
@Myastr0
Copy link
Owner

Myastr0 commented Dec 11, 2025

Hey @oliverkaiser, thanks for the ping, I worked the previous days on mk-notes to rework globally the architecture of the project and made it more stable for next updates/improvment like yours ! (cf #69)

Until I merge this global refactor, I'll use this PR to implement the --flatten option on the new system, sorry for the delay, and thank you again for your contribution !

@Myastr0
Copy link
Owner

Myastr0 commented Dec 14, 2025

Hey @oliverkaiser , I release a new version of mk-notes that include your (reworked) proposal in the v3.2.0.

You can check the documentation to know how to use the new --flat option which is highly inspired by this proposal !

@Myastr0 Myastr0 closed this Dec 14, 2025
@Myastr0 Myastr0 linked an issue Dec 14, 2025 that may be closed by this pull request
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.

Add ability to flatten pages

2 participants