The delete button used the pk for the phase and not the funding stream, so it led to a 404 not found.
I realized that the fix from PR #211 won't affect current scores. This fix works so it's backwards compatible, but should we maybe write a script to update the database and remove all None values from all ProjectScore rows?
Testing Instructions
visit review app tk
go to a phase
delete funding
make sure it redirects to a confirmation screen, and then back to the correct phase
Overview
Demo
Notes
The delete button used the pk for the phase and not the funding stream, so it led to a 404 not found.
I realized that the fix from PR #211 won't affect current scores. This fix works so it's backwards compatible, but should we maybe write a script to update the database and remove all
None
values from all ProjectScore rows?Testing Instructions