-
Notifications
You must be signed in to change notification settings - Fork 69
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
Content Audit: "address_2" in Sandy's DB #19379
Comments
Adding https://va-gov.atlassian.net/browse/VAHELP-8087 for @mmiddaugh as a new, additional reference that just came in this afternoon as well. |
Technical notes from the slack thread, to factor into next steps ticket
|
Asked LH if they validate against any special chars in |
Results of audit for VBAs
Several special characters were also found in address_1, plus 2 not seen in address_2:
|
Amazing, thanks MIchelle! Since Address_2 is used across all our facility types coming from LH, we will still need to do this audit for VAMC, Vet Center (and NCA? do we care about them yet?). In theory that's possible to do in the Drupal UI, but I still think a DB query will be the fastest way to get it done, which would be Drupal engineering work. |
Wait, I think I'm wrong, it's not possible to do in the Drupal UI, or in the Drupal DB. DaveP noted before: we smoosh address_1 and address_2 into a single Address field in Drupal, when we import. I think we have to ask LH to help with auditing address_2. @omahane does that sound right to you? |
(This ticket title is limited to Sandy's DB, but I don't think that was the spirit of trying to figure out special chars in address_2. I could be lost in the sauce.) |
Conversation with Adam re: address_2 in LH: https://dsva.slack.com/archives/C02BTJTDFTN/p1727970129537389 |
|
Sitewide can GET from LH API the contents of |
Michelle noted she can also pull addresses out of the VAST report. In that case, will include only have VHA + Vet Centers. Adam noted that for NCA: it depends if the cemetery is national or state.
|
https://dsva.slack.com/archives/C06EK4NLGDQ/p1729545788290989 here's the list of VAST locations (sharepoint). It includes VA health and Vet Center facilities (plus mobiles) which are marked as Active or Temp. Deactivated - consistent with LH API.
|
Things we need to figure out:
We need to do some smart thinking about how we might go about standardizing. This feels like a content modeling review, based on this audit. @davidmpickett does that sound right to you? If so, we could repurpose this ticket and refine it as content modeling, since the audit sort of happened async in the comments. |
@jilladams Content modeling seems incredibly premature here. That is a way of iterating on solution architecture. This feels like it is still early in Discovery. Here are some questions I would need answered before content modeling.
Address formatting and validation is also way more complex than phone number validation. Comparing this to the phone number standardization epic, that was a clearly defined subset of the phone number problem space: implement better guardrails in Drupal so that phone numbers are formatted consistently and can be fed in to the Design System component. We also already had a pattern in place that we were adopting, vs this where it’s not clear that formatting is even the issue |
Summary of further testing observations
About attached Data file
|
From refinement:
|
Clickthrough for "Get directions to Google maps" link in the month of September 2024
|
Update: We have identified and solved the problems with Guam, Puerto Rico and USVI in #19905. Still outstanding:
In this case we found that the Another contributing issue has been identified in the form of parameter-breaking characters in addresses being passed in URLs to Google Maps, specifically "&". While there are ways to defensively address this, it may not be worth the engineering effort. There are other situations where editors have just added too much information to the address_2 field (e.g. multiple buildings, floors, suites, etc.) and Google Maps is not able to effectively extract the target address. After speaking with @mmiddaugh at length, we are going to approach address issues identified in this ticket as follows: 1 - Michelle will look through Sandy's DB/VAST for addresses containing "&" and replace with the word "and" or reformat in another way. If we find that this is a widespread issue and not just a few instances, we can cut a ticket to replace "&" with "%26" in our links to Google Maps. This addresses the "&" issue. Related: The URL is sometimes generated with |
Background
The Google directions link is failing to find directions for some facilities that have data in Line 2 of the address. The assumption is that this results from data passed in "address_2" that contain certain special characters (which ones is currently unknown). In order to test, scope and evaluate possible solutions, we need to know the volume of special characters in in "address_2", across facility types.
Slack Thread
Technical notes
Drupal concatenates Address 1 & 2 into a single field when data is migrated in from Facilities API. Likely we will need to query the Facilities API, to get a complete sense of the various address_2 values.
Per Lighthouse (thread with Adam here), Lighthouse does not exclude any special characters from either
address_1
oraddress_2
.Some Google Directions links are based on:
While in theory it would save time to independently validate which of these characters causes the issue, there's also a case to be made that there could be others that are broken because of a different character being used. We can assume the address_2 field does not block any special characters, but Jill will reach out to LH to verify.
Acceptance Criteria
address_2
address_2
(not alpha, not numeric)The text was updated successfully, but these errors were encountered: