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

Script to Write Prereqs #872

Open
wants to merge 11 commits into
base: andrew--parse
Choose a base branch
from
Open

Conversation

andxu282
Copy link
Collaborator

@andxu282 andxu282 commented Nov 13, 2023

Summary

Adds firebase config file in python and creates a script to write prereqs to firestore.

Remaining TODOs:

  • Run script on dev
  • Run script on prod

@andxu282 andxu282 requested a review from a team as a code owner November 13, 2023 01:34
@dti-github-bot
Copy link
Member

dti-github-bot commented Nov 13, 2023

[diff-counting] Significant lines: 1407. This diff might be too big! Developer leads are invited to review the code.

Copy link
Collaborator

@zachary-kent zachary-kent left a comment

Choose a reason for hiding this comment

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

If it's WIP, you can also open as a draft

Copy link
Contributor

github-actions bot commented Nov 13, 2023

Visit the preview URL for this PR (updated for commit 229fa67):

https://cornelldti-courseplan-dev--pr872-andrew-write-prereq-jb2hce9b.web.app

(expires Thu, 14 Dec 2023 23:18:15 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6d4e0437c4559ed895272bbd63991394f1e0e933

@PabloRaigoza
Copy link
Contributor

Exciting to see the new implementation for storing pre-req courses into firebase. I think the code you made is great first step to expanding our code base with Python scripts to store, write, and read from firebase. This code will definitely be useful for future scripts to use as reference if we need to use Python again for firebase interactions. Great job!

@andxu282 andxu282 changed the title [WIP] Script to Write Prereqs Script to Write Prereqs Nov 15, 2023
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.

None yet

4 participants