If a board fails to save (e.g. because it is full) then there will be no value in the DataStore for some value of the clear count. The current HistoryBoard code just increments from 1 until it finds a key with no value and assumes this is the end of the history, so a board that fails to save will truncate that search.
Anyway this is unnecessary because the boards now store the current clear count board.ClearCount.Value so there is no need to infer it by looking for keys and values in the DataStore.
dmurfet
commented
2 years ago
If a board fails to save (e.g. because it is full) then there will be no value in the DataStore for some value of the clear count. The current HistoryBoard code just increments from
1until it finds a key with no value and assumes this is the end of the history, so a board that fails to save will truncate that search.Anyway this is unnecessary because the boards now store the current clear count
board.ClearCount.Valueso there is no need to infer it by looking for keys and values in the DataStore.