Skip to content

notionConfig file and Concurrent Vault Uploads - #6

Open
zedeff19 wants to merge 2 commits into
quanphan2906:masterfrom
zedeff19:master
Open

notionConfig file and Concurrent Vault Uploads#6
zedeff19 wants to merge 2 commits into
quanphan2906:masterfrom
zedeff19:master

Conversation

@zedeff19

Copy link
Copy Markdown

Hi there! 👋

I’ve made a couple of improvements that I think will make the project easier to maintain and a bit faster to use:

Central Notion Config:
I added a notionConfig.ts file at the base of the project. This lets you set the Notion API version in one place and use it wherever you need it (right now, it’s used in notion.ts). This should make it easier to update the API version in the future without hunting through the code.

Faster Vault Uploads:
I updated the vault uploading process to use Promise.all() for concurrent uploads. This means the whole vault can be uploaded in parallel instead of one file at a time, which should make things noticeably quicker, especially for larger vaults.

Let me know if you have any questions or if there’s anything you’d like me to tweak!
Thanks for your work on this project—I’m excited to contribute.

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.

1 participant