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

Few crashes #22

Open
MKSG-MugunthKumar opened this issue Sep 29, 2024 · 1 comment
Open

Few crashes #22

MKSG-MugunthKumar opened this issue Sep 29, 2024 · 1 comment

Comments

@MKSG-MugunthKumar
Copy link
Contributor

There are a few crashes in the app

  1. Trying to convert an audiobook to MP3 before it is downloaded.
  2. Trying to transcribe before a downloaded audiobook is converted to MP3

The app expects the flow to be
Authenticate Audible -> List Books -> Download Books -> Convert Books -> Transcribe Books -> Authenticate Readwise -> Post Highlights

If a step is missed, the app crashes.

Either we fix this or change the way the menu is displayed so that the crash is avoided.
For example: When the user runs the Convert Books command, show only the list of downloaded books or error out asking the user to download a book before proceeding.

Another option is to build a Django web UI and keep the command line interface only for developers.

Open to ideas/suggestions/pull requests!

@GGyll @JulianLagier

@GGyll
Copy link
Owner

GGyll commented Sep 30, 2024

Django UI sounds great and should be easy to implement.
For the CLI I think the easiest way is to dynamically show the help menu on each step like you suggested.

Are you able to work on the Django implementation?

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