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

Map BNF code changes to current codes #380

Merged
merged 29 commits into from Apr 3, 2017

Conversation

sebbacon
Copy link
Contributor

@sebbacon sebbacon commented Mar 24, 2017

These code changes are based on a list provided as a spreadsheet by NHS
BSA.

Fixes #214

Also fixes #253 and fixes #342

Before this is closed, we should also make a list of codes we know to have changed within a year, and supply them to BSA as a query.

These code changes are based on a list provided as a spreadsheet by NHS
BSA.
@sebbacon sebbacon added the now label Mar 24, 2017
These were expressed wrongly in the spreadsheet we were supplied.
This speeds things up mightily
Using this view should guarantee all codes are normalised to their
most recent representation.
Unfortunately necessary as a query and a view must both use the same SQL
flavour, and we use both because of different support for different
features in the respective flavours.
We might as well keep it in, as we'll only ever have to run any of this
command again if there's another major BNF release.
...because we don't currently import them
@sebbacon sebbacon merged commit 375e02a into master Apr 3, 2017
@sebbacon sebbacon removed the now label Apr 3, 2017
@sebbacon sebbacon deleted the map-historic-presentations-#214 branch April 3, 2017 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant