Skip to content
This repository has been archived by the owner on Aug 18, 2021. It is now read-only.

Scrub null json fields during harvesting #220

Open
froi opened this issue May 10, 2018 · 4 comments
Open

Scrub null json fields during harvesting #220

froi opened this issue May 10, 2018 · 4 comments

Comments

@froi
Copy link
Contributor

froi commented May 10, 2018

This issue references GSA/code-gov-web#560, created by @USDOJ-FSCP.

Original issue text:

At DOJ we are using a SharePoint solution that is not feature-rich when it comes to character level text processing. We are collecting code.json entries using an embedded forms feature in SharePoint, but we have limited ability to do anything that SharePoint doesn't provide some sort of plug in for. There is no readily available scripting tool within our reach to manage logic.

Because of that we have allocated a certain number of fields to various array values like keyword, language and so forth. So we have room on the UI for up to 5 keywords, and if a user only fills three of those blanks, the json file will have two null fields (,"") . This causes the harvester to post blank values, little null icons where the key words would be displayed. Its my thinking that a smarter harvester, one that discards null fields, would be a better investment than 10 - 15 improved json generating tools across the government that all will need to have logic that strips null fields.

@froi
Copy link
Contributor Author

froi commented May 11, 2018

I don't quite follow the issue. I would like to have a conversation about this to go over what exactly is the problem is.

CC. @RicardoAReyes @USDOJ-FSCP.

@USDOJ-FSCP
Copy link

USDOJ-FSCP commented May 11, 2018 via email

@froi
Copy link
Contributor Author

froi commented May 11, 2018

I've removed the phone numbers from your comment @USDOJ-FSCP You might want to delete the numbers from the signature when replying to emails from Github from your email client. These are public 😄

I'm at a conference but will carve out some time to make the call. If I can't I will post another comment.

@froi
Copy link
Contributor Author

froi commented Jul 31, 2018

@USDOJ-FSCP Sorry that this has fallen through the cracks. I'd like to retake this. I'll send you an email directly.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants