Skip to content

Create script to update geneds - #91

Open
nsandler1 wants to merge 35 commits into
masterfrom
gened-script
Open

Create script to update geneds#91
nsandler1 wants to merge 35 commits into
masterfrom
gened-script

Conversation

@nsandler1

@nsandler1 nsandler1 commented Feb 7, 2023

Copy link
Copy Markdown
Member

Closes #11

creates a management command to update geneds

@nsandler1 nsandler1 changed the title Gened script Create script to update geneds by semester Feb 7, 2023
@nsandler1 nsandler1 added the blocked Awaiting changes elsewhere label Feb 7, 2023
@Liam-DeVoe

Liam-DeVoe commented Feb 20, 2023

Copy link
Copy Markdown
Contributor

Does this command have to accept a semester? I was envisioning that this command simply defaulted to the latest semester. I can't think of any use case where we would want to use gened data from an old semester instead of the latest data available for all courses.

Hitting umdio without a semester parameter pulls the data for the most recent semester that course was taken.

@nsandler1

nsandler1 commented Feb 20, 2023

Copy link
Copy Markdown
Member Author

Does this command have to accept a semester? I was envisioning that this command simply defaulted to the latest semester. I can't think of any use case where we would want to use gened data from an old semester instead of the latest data available for all courses.

Hitting umdio without a semester parameter pulls the data for the most recent semester that course was taken.

I guess the semester argument doesn't need to be required but I want to keep it optional in case we want to run the script for a different semester

@nsandler1

Copy link
Copy Markdown
Member Author

This pr is ready for review again

@nsandler1 nsandler1 changed the title Create script to update geneds by semester Create script to update geneds Mar 8, 2023
Comment thread home/management/commands/updategeneds.py Outdated
Comment thread home/management/commands/updategeneds.py Outdated
Comment thread home/management/commands/updategeneds.py Outdated
@nsandler1

Copy link
Copy Markdown
Member Author

Blocked until semester comparator of umdio api is fixed

@nsandler1 nsandler1 added the blocked Awaiting changes elsewhere label May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked Awaiting changes elsewhere

Projects

None yet

Development

Successfully merging this pull request may close these issues.

update geneds

2 participants