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.

Sign up for GitHub

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

Jump to bottom

Add start chat button on models & assistants. #963

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add start chat button on models & assistants. #963

wants to merge 4 commits into from

Conversation

nsarrazin
Copy link
Collaborator

image

To help with #961, the idea is that when the model is active, instead of just showing "Active" we show a button you can click to start a chat

@nsarrazin nsarrazin added the front This issue is related to the front-end of the app. label Mar 29, 2024
@nsarrazin nsarrazin requested a review from gary149 March 29, 2024 08:31
@gary149
Copy link
Collaborator

gary149 commented Mar 29, 2024

It's a bit weird in its current state. As you need to click "activate" then "Start chatting" vs from the issue:

ok to replace "Activate" to "Start chatting" everywhere. (and it close the settings and it opens a new conv with the right model/assistant)

@nsarrazin
Copy link
Collaborator Author

Handled in 4554a0f !

Now it looks like this regardless of active status
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gary149 gary149 gary149 left review comments

Assignees
No one assigned
Labels
front This issue is related to the front-end of the app.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@nsarrazin @gary149