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

feat(participant): implement the docs command VSCODE-570 #817

Merged
merged 17 commits into from
Sep 19, 2024
Merged

feat(participant): implement the docs command VSCODE-570 #817

merged 17 commits into from
Sep 19, 2024

Conversation

alenakhineika
Copy link
Contributor

@alenakhineika alenakhineika commented Sep 16, 2024 •
edited
Loading

Description

Implement the @MongoDB /docs command that answers MongoDB-related questions and finds answers in the official documentation.

Currently, MONGODB_DOCS_CHATBOT_BASE_URI is provided as an env variable.

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Dependents

This PR should be updated with changes introduced in #810 when the PR is merged.

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@alenakhineika alenakhineika changed the title Vscode 570 docs command feat(participant): implement the docs command VSCODE-570 Sep 16, 2024
@alenakhineika alenakhineika marked this pull request as ready for review September 18, 2024 10:21
Anemy
Anemy reviewed Sep 18, 2024
View reviewed changes
package.json Outdated Show resolved Hide resolved
Anemy
Anemy reviewed Sep 19, 2024
View reviewed changes
Copy link
Member

@Anemy Anemy left a comment

Choose a reason for hiding this comment

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

Nice! Left a couple comments on things I noticed, I'll give it a manual test shortly.

src/participant/docsChatbotAIService.ts Outdated Show resolved Hide resolved
src/participant/participant.ts Outdated Show resolved Hide resolved
src/participant/participant.ts Outdated Show resolved Hide resolved
src/participant/participant.ts Outdated Show resolved Hide resolved
Anemy
Anemy approved these changes Sep 19, 2024
View reviewed changes
Copy link
Member

@Anemy Anemy left a comment

Choose a reason for hiding this comment

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

lgtm! Two small suggestions, not blockers.

src/participant/participant.ts Outdated Show resolved Hide resolved
src/test/suite/participant/participant.test.ts Outdated Show resolved Hide resolved
@alenakhineika alenakhineika merged commit e8b91be into VSCODE-528-mongodb-copilot Sep 19, 2024
3 checks passed
@alenakhineika alenakhineika deleted the VSCODE-570-docs-command branch September 19, 2024 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Anemy Anemy Anemy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alenakhineika @Anemy