gspread 6.0.0 - be careful about auto-archiver HOT 4 CLOSED

Thanks @djhmateer, pinning to 5.12.4 seems like a sensible idea until we know if there are any cases that are broken by gspread 6.0.0.

Looks like this change occurred here: burnash/gspread@6457676

from auto-archiver.

Pinning is a good decision for now.

Adapting to the new version should not be too hard and require changes only to https://github.com/bellingcat/auto-archiver/blob/main/src/auto_archiver/utils/gworksheet.py

namely

        if fresh:
            val = self.wks.cell(row, col_index + 1).value
            return val if val != "None" else ""

to keep consistency.

I'm not making the changes yet since testing if there were other changes to gspread.worksheet.get_values() to return "None"s too. But those are the only 2 places we interact with the gspread.worksheet to read the values.

from auto-archiver.

gspread devs were quick to acknowledge this and there will be a patch in 6.0.1: burnash/gspread#1403

from auto-archiver.

warning stands: do not use gspread 6.0.0, we can fix it at >=6.0.1 after that's released and tested.

from auto-archiver.