slight tweaks with devdb cos #2159
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
So per email with MC from DOB, it would be great if we could use CofO data from OpenData instead of a manual export that he emails us. Like applications and permits, there's one each for BIS and NOW
We can't actually use these yet - the NOW data doesn't have number of units, which we currently use from the custom export from MC. He says that they're working on getting those fields on OpenData. But since I wrote these, I figured it'd be nice to at least have these new ingest templates.
Also, while comparing data, I found that the logic we use to get devdb info from the c of o table was a little fussy, so I simplified the query. Tested this new query against a build on main
No rows returned
Build here