search

librariesio / libraries.io

:books: The Open Source Discovery Service
https://libraries.io
GNU Affero General Public License v3.0
1.12k stars 202 forks source link

Add Project.status_checked_at and populate this on Project#check_status #3112

Closed kjmahalingam closed 1 year ago

kjmahalingam commented 1 year ago

This adds a Project.status_checked_at field, so that we have a record of when Project#check_status was last called for a Project, without needing to use Project.updated_at to keep track of this.

kjmahalingam commented 1 year ago

For posterity's sake, does anyone know why I needed to add this commit? For some reason when I ran the migrations locally this was created in the schema, causing issues in the specs. I have no idea why it is trying to add this consistently when I run the migrations locally, given that I made no changes related to this.

EDIT: For those curious, the issue was resolved by updating scenic as the issue came from there. See here. Thanks @djpowers for the lead!

kjmahalingam commented 1 year ago

Broke the scenic upgrade out to #3113

kjmahalingam commented 1 year ago

@havocp @jhan217 I've added the index, let me know if there's anything you'd like me to implement differently there!

kjmahalingam commented 1 year ago

Alright again @havocp and @jhan217 I have fixed the index migration algorithm to be concurrent, should be ready now.