Skip to content

Make CliveDataTable manage the update of CliveDataTableRow (sync all with batch update)

Mateusz Żebrak requested to merge mzebrak/hp-info-table into develop

Some changes were made as a refactor to CliveDataTable.

I thought it would fix CliveDataTable "syncing rows separately on Hive Power Management" - because of batch_update usage. But seems like this is not the issue here. Giving it a batch update changes nothing, but as we later find out - the weird-looking behavior of HpInfoTable is probably not a bug because it's dependant on ongoing power-down, and it has to do with vests_to_hive conversion. Looks like due to the ratio dynamically being changed and ongoing conversion this is an expected behavior. The "fix" could be calculating the difference not on the Vests unit (like it's currently done) but on HP units, but this would be not so accurate. But that's some digression and this MR is just a refactor that I want to deliver so these "refactor" changes (in the right direction) won't be thrown out.

Merge request reports

Loading