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

SharePoint Plugin added #125

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

Conversation

kart2004
Copy link

Summary
This pr introduces a SharePoint data source plugin for RAGGENIE. The plugin enables users to connect and retrieve data from SharePoint within the platform.

Fixes #114

Changes Made

  • Plugin Initialization (init.py for SharePoint) Added a new SharePoint plugin file with connection arguments for site URL, client ID, client secret, and tenant ID.
    Structured the file to match the existing Website plugin to maintain uniformity across all data source plugins.
    Defined metadata and configuration settings specific to SharePoint.

  • Created a SharePointFormatter class to handle the formatting of data retrieved from SharePoint. The formatter processes the data based on inference details like the main entity and operation kind, ensuring that RAGGENIE can accurately present SharePoint data to users.

  • Created a handler file similar to the existing one for the Website plugin.

Developed the core SharePoint plugin class, including the following functionalities:

  • Connection Setup: Added methods to handle the connection to SharePoint using OAuth authentication parameters.
    Health Check: Implemented a health check method to verify the accessibility of the SharePoint site.
  • Data Retrieval: Integrated a method to fetch data from SharePoint using a base reader, ensuring compatibility with RAGGENIE's data processing architecture.
Untitled.video.-.Made.with.Clipchamp.mp4

:return: Data fetched from the SharePoint site.
"""
logger.info("Fetching data from SharePoint")

Choose a reason for hiding this comment

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

Implementation is not complete

site_url = self.params["site_url"]

try:
# Replace with actual SharePoint endpoint check if necessary

Choose a reason for hiding this comment

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

health check should also validate the credentials

@agberoz
Copy link
Collaborator

agberoz commented Oct 17, 2024

@kart2004, please make the changes

@ashmilhussain
Copy link
Contributor

@kart2004 are you working on this ?

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.

Add SharePoint Plugin
4 participants