Add zotero handler implementation

[ElinaKapetanaki]

Description

This is the Zotero Handler Implementation that is focused -for now- at connecting to the Zotero library of a user and selecting his/her annotations, which is highlighted text on research material (either all annotations, a specific one or all annotations on a specific item, like on a specific book). More information about the implementation can be found on the README file of the handler.

Also QA test is in this link showing that everything runs correctly https://github.com/ElinaKapetanaki/mindsdb-test/blob/main/Zotero_QA.MD

Fixes #5637

Type of change

• ⚡ New feature (non-breaking change which adds functionality)

Verification Process

To ensure the changes are working as expected:

• Test Location: Specify the URL or path for testing.
• Verification Steps: Outline the steps or queries needed to validate the change. Include any data, configurations, or actions required to reproduce or see the new functionality.

Checklist:

• My code follows the style guidelines(PEP 8) of MindsDB.
• I have appropriately commented on my code, especially in complex areas.
• Necessary documentation updates are either made or tracked in issues.
• Relevant unit and integration tests are updated or added.

[MinuraPunchihewa]

Hey @ElinaKapetanaki,
To continue our conversation on Slack here: is the pyzotero package installed in your virtual environment?

[ElinaKapetanaki]

Yes pyzotero is installed in my mindsdb-venv.

[mindsdbadmin]

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

[ElinaKapetanaki]

I have read the CLA Document and I hereby sign the CLA

[MinuraPunchihewa]

Hey @ElinaKapetanaki,
Thank you very much for this great contribution!
I've got a couple of requests:

1. Can you please resolve the conflicts here by merging the main branch into yours, please? The branches seem to be out of sync.
2. Can you please post a few screenshots or a quick video of this handler working as expected?

[ElinaKapetanaki]

Hello @MinuraPunchihewa ,
For the screenshots of the handler working as expected, in this link there are screenshots showing that everything works as expected : https://github.com/ElinaKapetanaki/mindsdb-test/blob/main/Zotero_QA.MD

Also I would like to post a screenshot of the tests for the handler passing as well:

[ElinaKapetanaki]

I also fixed the conflict problem.

[MinuraPunchihewa]

@ZoranPandovski Do you know the reason why this workflow is failing?

[ZoranPandovski]

That action deploys to our cloud dev environment by labeling the PR; we can ignore that. If this is ready, we can merge it tomorrow

[ElinaKapetanaki]

@ZoranPandovski What about the Test on Push failing with the error "Running on Github Actions but GITHUB_TOKEN is not set."?

[ElinaKapetanaki]

From my part, I think everything is ready and you can see it working in the link I have in above comments. This was my first ever PR, so I am very excited about merging tommorrow!! @MinuraPunchihewa @ZoranPandovski

[ElinaKapetanaki]

Goodmorning!! Are we merging this today? @MinuraPunchihewa @ZoranPandovski

[MinuraPunchihewa]

Can we please split ZoteroHandler and AnnotationsTable into their own modules? We can call them zotero_handler.py and zotero_tables.py. This will make it easier to add support for more tables (if any) moving forward?

[MinuraPunchihewa]

Can we please move this method to AnnotationsTable? Personally, I am not the biggest fan of including this function within the handler class, especially with the use of method_name parameter.

When moving it, since it does not require to be generic as such, we can maybe remove the method_name parameter.

[ElinaKapetanaki]

method_name from what I see can take 3 values : items, (to get all annotations) item (to get 1 annotation) or children (to get the annotations of for ex. a specific article), because these are 3 methods that are used from the API . So I am not sure if we can remove it, since that information must be passed to the method.

[MinuraPunchihewa]

Makes sense. Instead of removing it, can we then give it a more meaningful name? When moving it to the Table class, I think you could even break it down into three functions like get_items(), get_items() and get_item_children().

The approach to take there is completely up to you; my only request is that we move this method to the class where the logic belongs.