-
Notifications
You must be signed in to change notification settings - Fork 4
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
Allow Importing of CSV Files To Plan #83
base: staging
Are you sure you want to change the base?
Conversation
Duplicates are expected, all the import is doing is taking the courses from the csv and adding them to the plan, The csv doesnt have any information about what underlying choices resulted in those courses, so I cant remove them from the requirement bar. Perhaps I should give more detail as to how the import works in the tab description |
Yeah I think everything is working correctly functionality-wise other than those two; adding an explanation to the import page should be good. |
What are we doing with this? |
Tested, CSS looks really clean! I noticed the background color of the sign out button isn't the same as the add courses (it is actually #f5f5f5, not white). I have already pushed a commit to address this. Good to merge to prod. @b47huang Please take a look at it as the last pair of eyes. |
UPDATE: This feature was replaced by the Auth system. I am keeping this PR around in case we still want to add something like this later.
Adds a new tab which gives a brief explanation to the user and lets them upload csv files.
The courses will be read from the csv and added to the plan.