🚀 Major Enhancements and Modernization for enex2notion#123
Open
subimage wants to merge 12 commits intovzhd1701:masterfrom
Open
🚀 Major Enhancements and Modernization for enex2notion#123subimage wants to merge 12 commits intovzhd1701:masterfrom
subimage wants to merge 12 commits intovzhd1701:masterfrom
Conversation
This was referenced Jun 6, 2025
|
It works. Thanks for your update! |
|
is this dead? having issues with the current version and think this version would solve. |
jimeh
reviewed
Dec 31, 2025
|
|
||
| error_message = f"Failed to get notebook root for '{notebook_title}'" | ||
| get_func = get_notebook_database if self.mode == "DB" else get_notebook_page | ||
| get_func = get_notebook_page if self.mode == "DB" else get_notebook_page |
There was a problem hiding this comment.
question: Is the complete removal of DB mode support intentional? There's no mention of it in the PR or README changes, and --help output still shows the --mode option, stating default is `DB.
There was a problem hiding this comment.
I see, is this why I can't import my notebook as a database? Is there a fix for this?
|
The work in this PR is no small feat 👏 I have gotten it to work, but also run into a couple of issues:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚀 Major Enhancements and Modernization for enex2notion
Overview
This PR brings significant modernization and performance improvements to enex2notion, including Python 3.13 support, Notion API v2 migration, concurrent processing, and robust error handling with resume functionality.
🔧 Key Improvements
1. Python 3.13 & Dependency Modernization
PyMuPDF→ v1.24.11+ (fixes Python 3.13 build issues)lxml→ v5.2.2+ (Python 3.13 compatible)notion-client→ v2.3.0+ (modern Notion API)2. Notion API v2 Migration & Authentication
3. Performance & Concurrency
4. Robust Error Handling & Resume Functionality
[UNFINISHED UPLOAD]markers - easily identify partial uploads5. Enhanced Content Processing
6. Code Quality & Maintainability
🐛 Bug Fixes
📊 Performance Impact
🔄 Breaking Changes
📚 Documentation Updates
🧪 Testing
Why This Matters
The original enex2notion project hasn't been updated since October 2023 and doesn't work with:
This fork makes the tool production-ready for 2025 and beyond, ensuring users can reliably migrate their Evernote data to Notion with modern Python environments.
Ready to merge? This represents several days of development work to modernize and significantly improve the reliability and performance of enex2notion. All changes are backward-compatible where possible, with clear migration guidance for breaking changes.