search

buchhalter-ai / buchhalter-ai-cli

The buchhalter open-source command line tool to automate invoice downloads from suppliers. Free, customizable, and designed for seamless financial workflow integration.
https://buchhalter.ai
Other
4 stars 1 forks source link

`sync` cmd: Move init operations into command logic to start Bubbletea application earlier #108

Closed andygrunwald closed 1 month ago

andygrunwald commented 1 month ago

The sync cmd initialized many objects and ran a few operations before the bubble tea application started. This had the effect that the user received a 1Password prompt before any application output was done. UI/UX wise, this seems strange:

  1. You start the app
  2. No output
  3. 1Passwort wants you to authenticate

After the change, the buchhalter logo is shown and a message is shown that you try to initialize the 1Passwort Vault provider.

Additionally, many more operations are shown in the UI as successful.

Screenshot 2024-09-15 at 22 00 49

Merge order

This PR depends on https://github.com/buchhalter-ai/buchhalter-ai-cli/pull/107 https://github.com/buchhalter-ai/buchhalter-ai-cli/pull/107 need to be merged before.