I don't think this covers #73 completely, but implementing the full solution would take a lot of modification to the code, and I don't think we need it right now.
As a temporary solution to the actual problem in #73, we have a update_last function that only update the fields that we want to update from helper, it basically hides the fact we update the target directly, and makes it a little better in catching that we only need to update certain fields instead of the whole target struct.
stiiifff
commented
4 months ago
I don't think this covers #73 completely, but implementing the full solution would take a lot of modification to the code, and I don't think we need it right now.
As a temporary solution to the actual problem in #73, we have a
update_lastfunction that only update the fields that we want to update from helper, it basically hides the fact we update the target directly, and makes it a little better in catching that we only need to update certain fields instead of the whole target struct.