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.

Sign up for GitHub

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

Jump to bottom

[Property] LocPopNumber no longer needed after merge with property loc file #185

Closed
johcarter opened this issue May 23, 2024 · 1 comment
Closed

[Property] LocPopNumber no longer needed after merge with property loc file #185

johcarter opened this issue May 23, 2024 · 1 comment
Assignees
@MattDonovan82
Labels
major major release (breaking change)

Comments

@johcarter
Copy link
Contributor

@stufraser1 can you confirm whether the 'LocPopNumber' field is still needed to be included in the spec? Now the LocPopulation file is part of the OED loc file, this looks likes a hangover from when the files were separate that linked the population to the property data? Now they are integrated, does the existing LocNumber field suffice?

Originally posted by @MattDonovan82 in #40
@johcarter johcarter moved this to Under discussion in Open Data Standards May 23, 2024
@MattDonovan82 MattDonovan82 self-assigned this May 23, 2024
@johcarter johcarter changed the title LocPopNumber no longer needed after merge with property loc file [Property] LocPopNumber no longer needed after merge with property loc file May 24, 2024
@johcarter johcarter moved this from Under discussion to In progress in Open Data Standards Jun 4, 2024
@johcarter johcarter moved this from In progress to In review in Open Data Standards Jun 20, 2024
@johcarter johcarter moved this from In review to Under discussion in Open Data Standards Jun 25, 2024
@johcarter johcarter added the major major release (breaking change) label Jun 25, 2024
@johcarter johcarter moved this from Under discussion to Agreed and ready in Open Data Standards Jun 26, 2024
@johcarter johcarter moved this to Under discussion in OED 4.0.0 Jun 26, 2024
@johcarter johcarter moved this from Under discussion to Agreed and ready in OED 4.0.0 Jun 26, 2024
@johcarter johcarter moved this from Agreed and ready to In progress in OED 4.0.0 Jun 26, 2024
johcarter added a commit that referenced this issue Jun 26, 2024
@johcarter
#187 #186 #162 #185
53a98de
@johcarter johcarter moved this from In progress to In review in OED 4.0.0 Jun 26, 2024
@MattDonovan82
Copy link
Contributor

I don't think the LocPopNumber is still required in the spec. Agree to remove this field as LocNumber will suffice. @stufraser1 ?

@MattDonovan82 MattDonovan82 moved this from In review to Agreed and ready in OED 4.0.0 Jul 11, 2024
@johcarter johcarter moved this from Agreed and ready to In review in OED 4.0.0 Jul 11, 2024
@MattDonovan82 MattDonovan82 moved this from In review to Done in OED 4.0.0 Jul 11, 2024
@MattDonovan82 MattDonovan82 moved this from Todo to In Progress in Oasis Dev Team Tasks Jul 12, 2024
@johcarter johcarter moved this from In Progress to Waiting for Review in Oasis Dev Team Tasks Jul 17, 2024
sambles pushed a commit that referenced this issue Jul 17, 2024
@johcarter @sambles
#187 #186 #162 #185
bc06263
sambles pushed a commit that referenced this issue Jul 17, 2024
@johcarter @sambles
#187 #186 #162 #185
442eeb0
@johcarter johcarter mentioned this issue Aug 21, 2024
Merged
@benhayes21 benhayes21 moved this from Waiting for Review to Done in Oasis Dev Team Tasks Sep 4, 2024
@github-project-automation github-project-automation bot moved this from Agreed and ready to Done in Open Data Standards Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MattDonovan82 MattDonovan82

Labels
major major release (breaking change)
Projects
OED 4.0.0
Status: Done
Oasis Dev Team Tasks
Status: Done
Open Data Standards
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@johcarter @MattDonovan82