-
Notifications
You must be signed in to change notification settings - Fork 0
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 bulk export functionality to DefraRuby::Exporters #105
Conversation
https://eaflood.atlassian.net/browse/RUBY-68 The second half of the functionality for the DefraRuby Exporters library is to add bulk exports. This is the feature to generate CSV files containing all of the registration exemptions data in the database.
81bdde1
to
8ca99b6
Compare
The force push was to fix a typo in the commit message. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of the feedback is personal opinion so open for discussion.
The one overarching thing I'd say is if I'd have taken the time to understand how complex wrangling the date ranges would be and how much code it would add, I might have gone for something more hacky and far less refined as our starter for ten. But that's a lesson for me, and the upside is we should be able to handle anything thrown at us now!
https://eaflood.atlassian.net/browse/RUBY-68
The second half of the functionality for the DefraRuby::Exporters library is to add bulk exports. This is the feature to generate CSV files containing all of the registration exemptions data in the database.