Skip to content

feat: multiple agent support #59

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

Merged
merged 6 commits into from
Apr 28, 2023
Merged

feat: multiple agent support #59

merged 6 commits into from
Apr 28, 2023

Conversation

ytcheng
Copy link
Contributor

@ytcheng ytcheng commented Feb 13, 2023

add multiple agent support
Screenshot
1676292037899

Copy link
Member

@kylecarbs kylecarbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution! 🥳

Some lines have ; and some don't. If you could run the prettier formatter, that'd be great! Have you manually tested this? I'll give it a shot in a bit too.

@ytcheng
Copy link
Contributor Author

ytcheng commented Feb 14, 2023

Thank you for the contribution! 🥳

Some lines have ; and some don't. If you could run the prettier formatter, that'd be great! Have you manually tested this? I'll give it a shot in a bit too.

Yes, I've manually tested. It works

@ytcheng ytcheng requested a review from kylecarbs February 16, 2023 07:07
@kylecarbs
Copy link
Member

@ytcheng I'll test, merge, and do a release in the morning (it's 1am here) ;p.

Thanks again!

@smolinari
Copy link

Just a small improvement suggestion and a question.

The text should be "Connect to an Agent" (a to an). 😁

Also, what would be multiple agents? In a workspace?

Scott

@kylecarbs
Copy link
Member

@smolinari some users will have a workspace that contains a database and a container that they code in, or maybe multiple containers in more abstract scenarios.

In these situations, you might have multiple agents you can connect to inside a single workspace!

@smolinari
Copy link

Hmm... interesting. Didn't think about having more than one agent in a workspace. Cool. More possibilities.... 😁

Scott

@bpmct bpmct requested review from rodrimaia and kylecarbs and removed request for kylecarbs April 10, 2023 18:19
@rodrimaia
Copy link
Contributor

✅ Updated branch with most recent changes
✅ Added support to also selecting agents on Sidebar:

When the workspace only has multiple agents, now we list them and allow users to open an agent directly:
image

For workspaces with a single agent, use the default view:
image

@rodrimaia rodrimaia merged commit 4662ebd into coder:main Apr 28, 2023
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

Successfully merging this pull request may close these issues.

4 participants