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

Feature/Outpost-92 genericise codebase #248

Merged
merged 5 commits into from
Aug 16, 2022
Merged

Conversation

em-cd
Copy link
Contributor

@em-cd em-cd commented Aug 10, 2022

This was a lot easier than I thought it would be!

There was only really the import folders left that referenced BOD/BFIS. I've moved everything into the generic data_import folder and added descriptions for the tasks so they show up in the task list (when you run rake --tasks or rake -T), but apart from tweaking the imported file names, I have not touched the code. I suggest we don't worry about making sure it all works and is documented until we get to needing to do an import again, because otherwise we might do it all now and then need to change it all again once it comes to actually using it.

em-cd added 5 commits August 10, 2022 16:35
Some of these import tasks may no longer be needed, or need to be
documented. For now this commit moves all of the old files out of the
BOD & BFIS folders so that we no longer have hard-coded references to Bucks-specific
tasks.
@em-cd
Copy link
Contributor Author

em-cd commented Aug 10, 2022

Oh yes and the app links still reference Bucks - but we have a separate ticket for that so not tackling that here 😄

@apricot13 apricot13 self-requested a review August 16, 2022 09:08
@em-cd em-cd merged commit 4e8ca4d into develop Aug 16, 2022
@em-cd em-cd deleted the feature/outpost-92-genericise branch August 16, 2022 09:36
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