-
Notifications
You must be signed in to change notification settings - Fork 92
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
More automated process for indicator status updates #854
Comments
After Jen (I guess I'll be doing them now) does pull requests, I go onto the staging site and check for a few things: data complete?, graph o.k.?, metadata updated? anything else pending?... assuming all goes well, I then make a list of indicators that should receive "stars" ... Jen or Angela would manually put it in (not sure if it is via editing the code) to show the indicator is done updating for this round. What I was thinking-- similar to the existence of an URL in the metadata triggering a graph, can we create an entry in the metadata (e.g. Is update complete? Y/N) to "toggle" the star? Thanks for looking into this. |
@Kali2017SDG I've given this a try on #855. I'm not sure of an easy way to test prose.io fields, but in theory this should add a checkbox for "Is update complete?" when you're editing the metadata. |
@Kali2017SDG @philipashlock Just noting a recap of our discussion today about this:
|
Looping in @SmithersA. Angela, see some notes above about trying to automate the display of the "star" next to indicators that are "complete" after a data call. |
@brockfanning & @Kali2017SDG ~ probably added the "updated" tag regardless of working graph (because sometimes there wasn't a graph) |
I have a demo of using the logic to key off of a This will require a well formatted date (e.g. There may be more automated ways (with a plugin or pre-commit script) that we can explore so this field doesn't have to be manually edited, but I'm not sure if those solutions will be available to us with the current workflow. We could also use javascript to calculate the last six months date rather than calculate using the last site build date. |
This seems like a good direction to me. Would a reasonable next step be to write a one-off script that tries to convert all the invalid |
I think we may need to do some manual review and updates that wouldn't be super easy to do with a script unless you wanted to query the GitHub API for commit dates. I don't think we have a lot to update though so I figured I'd create a new view to better understand where we are. The new table view here lists all indicators, their status, and the last updated date entered into the metadata. Here's how it works:
If the star image is present, the text "STAR" is present, and the date is present then we probably don't need to make any change. The one thing we might want to update is if the date is on the first of the month, since that implies that a specific day of the month wasn't specified for the date_metadata_updated and we should update the date with the exact date in the standardized format. 1.2.1 is an example of this situation. If the star image is present and the text "STAR" is not present it either means no valid date was provided and we'll need to update the metadata with date_metadata_updated in the standardized format - or if a good date was provided but is not within the past 6 months then everything is behaving as we expect and the star to no longer be visible when we roll out this new automated approach. 3.4.2 is an example where no valid date was provided and 3.c.1 is an example where a date was provided, but it's more than 6 months ago. If the star image is not present and the text "STAR" is present that should mean that the star image has just not been manually added for this indicator, but we shouldn't need to take any action since it will be displayed automatically when we roll out this new automated approach. 1.5.3 is an example of this scenario. So really the main thing we want to review here is to make sure that we have a good date entered for date_metadata_updated on anything that's been updated in the past 6 months (ideally for everything, but the priority will be for all changes we want to use to trigger this star to display). So any place in this table where there's a star image, but no "STAR" text and no date listed is an example where we'll definitely need to add a date in the metadata, e.g. 3.1.2, 3.2.1, 3.2.2, 3.4.2, etc. |
@philipashlock @Kali2017SDG @SmithersA Who would know the correct dates to add for the indicators in question? (3.1.2, 3.2.1, 3.2.2, 3.4.2, etc.) Or should we just use the last time the file was changed in Github? Also meanwhile, I've submitted a pull request that implements date validation for |
@Kali2017SDG You mentioned it would be helpful to get more automation into the platform regarding indicator status updates, I believe. To kick things off, could you describe the manual process that you're currently doing? For example, what URL do you go to, what do you click on, etc.
The text was updated successfully, but these errors were encountered: