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

Unit: N/A, NA, etc. #44

Open
bertday opened this issue Nov 28, 2016 · 2 comments
Open

Unit: N/A, NA, etc. #44

bertday opened this issue Nov 28, 2016 · 2 comments
Labels
enhancement

Comments

@bertday
Copy link
Contributor

bertday commented Nov 28, 2016

We received some feedback from a Property Search user who didn't get any results because they entered NA for the unit. Should these be stripped out by Passyunk?

  • N/A
  • NA
  • None
  • unknown
  • etc.
@tswanson
Copy link
Contributor

I've been monitoring the 404 returns for AIS and this is the first time I've seen this. Property needs a single field entry.

@bertday
Copy link
Contributor Author

bertday commented Dec 1, 2016

Agree, single field would help this. But I could still see it coming over in voters or something other manually-entered dataset. Just something to think about.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bertday @tswanson