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

[GoEx] Added additional support for GoEx DBs and APIs. #698

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

Conversation

YLu-1258
Copy link

GoEx Db

  • Created additional DBManagers for PostgreSQL and MongoDB databases.
  • Implementation for MongoDB is limited as it is a document-based db, rather than SQL.
    • Included sample driver code to test MongoDB operations.

GoEx APIs

  • Utilized the Spotipy module to create basic APIs to spotify (album and playlist details)

Co-authored by: Yujie Lu (lusane24@berkeley.edu)

@HuanzhiMao HuanzhiMao changed the title Added additional support for GoEx DBs and APIs. [GoEx] Added additional support for GoEx DBs and APIs. Oct 14, 2024
Copy link
Owner

@ShishirPatil ShishirPatil 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 PR @YLu-1258 ! Similar to the previous comment, please also do update the readme - on how we can test these out, so a) we can test it out locally, and b) it will be useful to users who want to use this.

@YLu-1258
Copy link
Author

YLu-1258 commented Nov 8, 2024

Thank you for the PR @YLu-1258 ! Similar to the previous comment, please also do update the readme - on how we can test these out, so a) we can test it out locally, and b) it will be useful to users who want to use this.

Hello Mr. Patil!

Sorry for the delayed response, my partner and I have been preoccupied with midterms season recently and have just finished. Please find my new commits for how to use and test gorilla with PostgreSQL and MongoDB up above!

@lusane24
Copy link

Hello Mr. Patel,

Sorry for the very late response! Alex and I have been very busy with midterms and we have just finished. I have updated the read me with steps on how to test the functions for the Spotify APIs implemented.

Thank you for your patience and understanding,
Yujie Lu

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.

3 participants