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

SPD fix #578

Merged
merged 2 commits into from
Aug 23, 2020
Merged

SPD fix #578

merged 2 commits into from
Aug 23, 2020

Conversation

cblanc
Copy link
Member

@cblanc cblanc commented Aug 23, 2020

Some postcodes on SPD have an extra character appended. Since these are
picked up as invalid postcodes, they can no longer be queried

This change will ingest the first instance of these invalid postcodes
(i.e. those ending with A) and drop all others

Fixes #577

cblanc added 2 commits August 23, 2020 15:32
SPD have appended extra characters to a small number of postcodes -
causing those postcodes to be unqueryable
Some postcodes on SPD have an extra character appended. Since these are
picked up as invalid postcodes, they can no longer be queried

This change will ingest the first instance of these invalid postcodes
(i.e. those ending with A) and drop all others
@codecov
Copy link

codecov bot commented Aug 23, 2020

Codecov Report

Merging #578 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #578      +/-   ##
==========================================
+ Coverage   98.01%   98.03%   +0.01%     
==========================================
  Files          40       40              
  Lines        1312     1320       +8     
==========================================
+ Hits         1286     1294       +8     
  Misses         26       26              
Impacted Files Coverage Δ
app/models/scottish_postcode.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1eab750...954469c. Read the comment docs.

@cblanc cblanc merged commit 86de005 into master Aug 23, 2020
@cblanc cblanc deleted the spd_fix branch August 23, 2020 15:55
@idealbot
Copy link
Member

🎉 This PR is included in version 12.0.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

cblanc added a commit that referenced this pull request Dec 17, 2020
Reintroduces fix from #578

Some postcodes on SPD have an extra character appended. Since these are
picked up as invalid postcodes, they can no longer be queried

This change will ingest the first instance of these invalid postcodes
(i.e. those ending with A) and drop all others
cblanc added a commit that referenced this pull request Dec 17, 2020
Reintroduces fix from #578

Some postcodes on SPD have an extra character appended. Since these are
picked up as invalid postcodes, they can no longer be queried

This change will ingest the first instance of these invalid postcodes
(i.e. those ending with A) and drop all others
idealbot pushed a commit that referenced this pull request Dec 17, 2020
## [13.0.2](13.0.1...13.0.2) (2020-12-17)

### Bug Fixes

* **SPD:** Add back dual SPD entries ([80f2f90](80f2f90))
* **SPD:** Correct and ingest invalid SPD postcodes ([8564f5a](8564f5a)), closes [#578](#578)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

Scottish Postcode Directory
2 participants