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.

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

added discord connector #783

Open
wants to merge 30 commits into
base: develop
Choose a base branch
from

Conversation

SubramanyamChalla24
Copy link

@SubramanyamChalla24 SubramanyamChalla24 commented Oct 14, 2023

Branching

Always branch off from the develop branch with a descriptive name that encapsulates your idea or fix. This practice helps ensure that our main branch remains stable.

Description

Added the discord connector

Related Issue(s)

#689

Testing and Validation

Tested this only on local system as the discord connector doesn't seem show up in the admin dashboard when using docker. Would require a test from others and verify.

Screenshots (if appropriate):

image

Documentation

Changes Include

  • Bugfix (non-breaking change that solves an issue)
  • New Feature / Enhancement (non-breaking change that add or improves functionality)
  • New Feature (breaking change that is not backwards compatible and/or alters current functionality)
  • Documentation (change to product documentation or Readme.md only)

PR Checklist

  • This PR targets the develop branch of the swirl-search repo.
  • The above portion of this template is filled out completely.
  • [] This PR has been tested for correct functionality and validated not to break existing product code.

swirl/connectors/discord.py Outdated Show resolved Hide resolved
swirl/connectors/discord.py Outdated Show resolved Hide resolved
Copy link
Contributor

@sid-swirl sid-swirl left a comment

Choose a reason for hiding this comment

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

@SubramanyamChalla24 this looks awesome you have made a lot of progress; please review the comments I added to the "files changed" and can discuss on Slack. It's very close!

swirl/connectors/discord.py Outdated Show resolved Hide resolved
@SubramanyamChalla24
Copy link
Author

Hi, I've added a connector for Qdrant as mentioned in the issue #721 .

Copy link
Contributor

@sidprobstein sidprobstein left a comment

Choose a reason for hiding this comment

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

See comment - thank you!!

requirements.txt Outdated Show resolved Hide resolved
@sidprobstein sidprobstein mentioned this pull request Oct 29, 2023
@sidprobstein
Copy link
Contributor

@SubramanyamChalla24 please make one final change:

  • Put the SearchProviders in SearchProviders/untested
  • Put the Connectors in swirl/connectors/untested

Omit changes to init.py for now. Thank you! I will merge immediately once these changes are made.

@heurainbow
Copy link

what is the status now, qdrant and discord? pending over a month now

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.

5 participants