radgrad / radgrad2

Source code for RadGrad, Version 2
https://radgrad.org
9 stars 5 forks source link

Update checkIntegrity for profile collections #775

Closed philipmjohnson closed 2 years ago

philipmjohnson commented 3 years ago

Please do work for this task in a branch called issue-775.

It appears that check integrity does not flag opportunities in the profile that have subsequently been deleted. Fix this so running the integrity check reveals this problem.

cammoore commented 2 years ago

Do you mean retired or deleted?

philipmjohnson commented 2 years ago

Hmm. I think deleted? It's not an integrity violation to link to a retired opportunity, I think, because that's why we have the "retired" flag (i.e. it won't be presented to new users, but existing links to it are valid.)

Does that make sense?

cammoore commented 2 years ago

Duplicate. Already fixed.