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

genanki.Note should have a field for the paths to its media files #60

Open
kerrickstaley opened this issue Nov 29, 2020 · 1 comment
Open

Comments

@kerrickstaley
Copy link
Owner

kerrickstaley commented Nov 29, 2020

Instead of having a .media_files attribute on Package, it may be more ergonomic to have a .media_files attribute on each genanki.Note instance. The .media_files for the overall package will then be computed by combining the lists for each individual Note (removing duplicates).

Need to think a little more about whether this design will handle all use-cases and whether it will be genuinely easier to use. One consideration is "static" media files; those still need to be specified at the package level.

@yash-fn
Copy link

yash-fn commented Feb 3, 2022

I vote keeping it the way that it is now. The same media file can be referenced in many different ways. Might as well save yourself a headache engineering something so specific in and force the user to make sure they know what they want when specifying the media writes. The greatest utility your module provides is not having to manually generate that sqlite db. KISS where possible.

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

No branches or pull requests

2 participants