marvin icon indicating copy to clipboard operation
marvin copied to clipboard

If I already have the default bots installed, the TUI should not have an install button

Open discdiver opened this issue 2 years ago • 2 comments

First check

  • [X] I added a descriptive title to this issue.
  • [X] I used the GitHub search to look for a similar issue and didn't find it.
  • [X] I searched the Marvin documentation for this feature.

Describe the current behavior

I already have the default bots installed. I click on "Bots". The TUI has a "Install default bots" button, which is confusing. A user might think the bots are not installed and that they need to install each bot individually.

Screenshot 2023-04-21 at 10 42 58 PM

Describe the proposed behavior

The "Install Default Bots" button should only show if the default bots are not installed.

Example Use

No response

Additional context

No response

discdiver avatar Apr 22 '23 04:04 discdiver

Hi, i could work on this if that's okay.

nickkatsios avatar Apr 23 '23 12:04 nickkatsios

One issue here is that we don't currently have a concept of what a "default" bot is (or a "bot library"), so determining whether all of the bots exist and are unmodified is an open question

jlowin avatar Apr 23 '23 15:04 jlowin