OpenNewsLabs / autoEdit_2

Fast text based video editing, node Electron Os X desktop app, with Backbone front end.
https://opennewslabs.github.io/autoEdit_2/
MIT License
420 stars 56 forks source link

No way to change IBM credentials #35

Closed kristinalustig closed 7 years ago

kristinalustig commented 7 years ago

If a user enters incorrect or invalid IBM credentials there is no way to remove them and enter updated credentials - even after deleting and reinstalling the mac app.

pietrop commented 7 years ago

Hi @kristinalustig, Yes, that's a good point, and something I am planning to address. Perhaps by adding a settings panel, where the user can review the credentials.

Here's a bit of an update of where I am at with this.

At the moment, if you uninstall using app cleaner, as described in the user manual when re-installing again it will prompt you to add the credentials again. As app cleaner does a clean uninstall. while just dragging the icon into the trash leaves all the system preferences on the computer (including the API credentials).


This is a bit of a hack, but to review credentials, they are stored inside

/Users/{{you username for the home folder}}/Library/Application Support/autoEdit2/wttskeys.json

To get to that file:


However I'd normally just recommend to uninstall and try again. I am also working on adding some better Error handling to signal the issue when a user puts incorrect credentials and IBM watson returns an error.

pietrop commented 7 years ago

connected with this issue #4

pietrop commented 7 years ago

Added error handing when user puts wrong/incorrect credentials for IBM Watson stt. This will be updated in next release in version 1.0.6

screen shot 2017-04-28 at 00 34 55

For clarities purposes, and in case anyone has better suggestions on how to best implement this, Here is an overview of the implementation.

Modified these modules responsible for generating an saving the transcription

More error handling coverage is needed across the app but this is a start.

pietrop commented 7 years ago

Closing for now. Going to deal with allowing user to edit/change credentials with a settings panel at a later stage as a separate issue.