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

Add courses exporter #197

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

eveninglily
Copy link
Member

Take the exporter that is in umdio-data, clean it up a bit, and add it here.

Add a rakefile task to run it

Take the exporter that is in umdio-data, clean it up a bit, and add it here.

Add a rakefile task to run it
@tybug
Copy link
Contributor

tybug commented Apr 18, 2021

Am I missing something or uh, did the actual export/courses.rb file not get added to this pr?

@DonIsaac
Copy link
Member

@McIntireEvan should we have test coverage for this?

app/export/courses.rb Show resolved Hide resolved
@eveninglily
Copy link
Member Author

@McIntireEvan should we have test coverage for this?

I can add a test that imports a sem, then exports it and ensures they match

@DonIsaac
Copy link
Member

@McIntireEvan any updates?

@DonIsaac DonIsaac self-requested a review April 26, 2021 23:33
@DonIsaac
Copy link
Member

DonIsaac commented May 7, 2021

@McIntireEvan pinging you again to get the status of this PR

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

3 participants