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

Schedule Generator Front/Back End Connection #916

Conversation

nidhi-mylavarapu
Copy link
Contributor

@nidhi-mylavarapu nidhi-mylavarapu commented Apr 15, 2024

Summary

This pull request is for connecting the schedule generator algorithm with the sched gen frontend components (still in progress)

  • Passed in relevant functions / objects
  • Replaced some hardcoded data

Remaining TODOs:

  • Properly pass in class information (first with a test request, then with user inputs), see if generated schedule is valid / properly formatted

Test Plan

@nidhi-mylavarapu nidhi-mylavarapu requested a review from a team as a code owner April 15, 2024 02:36
@dti-github-bot
Copy link
Member

dti-github-bot commented Apr 15, 2024

[diff-counting] Significant lines: 414.

Copy link
Contributor

github-actions bot commented Apr 15, 2024

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

https://cornelldti-courseplan-dev--pr916-nidhi-sched-gen-fron-l23cwe9e.web.app

(expires Sun, 19 May 2024 16:07:32 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6d4e0437c4559ed895272bbd63991394f1e0e933

@andxu282 andxu282 force-pushed the sched-gen-courses-frontend branch from 3f9e26e to 46aa4a7 Compare April 15, 2024 22:36
@andxu282 andxu282 changed the base branch from sched-gen-courses-frontend to schedule-generator April 18, 2024 22:19
@andxu282 andxu282 force-pushed the nidhi/sched-gen-frontend-backend-connection branch from fe4503f to 5ec4a64 Compare April 18, 2024 22:40
@nidhi-mylavarapu nidhi-mylavarapu force-pushed the nidhi/sched-gen-frontend-backend-connection branch from ec9c7e2 to 907f6ef Compare April 20, 2024 17:52
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.

2 participants