search

move-coop / parsons

A python library of connectors for the progressive community.
Other
254 stars 125 forks source link

Adding a data paste function to Google Sheets #1045

Closed anzelpwj closed 1 month ago

anzelpwj commented 2 months ago

Hi all,

Following up on issue #1036 . I tried initially modifying the append_to_sheet function, but it became clear that this is simply asking too much for one function, so I made a new method paste_data_in_sheet instead.

I don't have access to the Google Sheets sandbox, so for now test_paste_data_in_sheet has not been confirmed to work as expected (but I have done some testing with my own spreadsheet).

Any advice on how I've designed this/getting this code ready for merging would be much appreciated, thank you!

anzelpwj commented 2 months ago

Why are imports failing for modules I haven't touched?

NirTatcher commented 2 months ago

Why are imports failing for modules I haven't touched?

What version of Python are you running? I get the same error when I'm on an incompatible python version , make sure it matches the current python version requirements: python_requires=">=3.7.0,<3.12.0"

Hope this would be helpful!

anzelpwj commented 2 months ago

Why are imports failing for modules I haven't touched?

What version of Python are you running? I get the same error when I'm on an incompatible python version , make sure it matches the current python version requirements: python_requires=">=3.7.0,<3.12.0"

Hope this would be helpful!

It runs fine on my machine (Pop!_OS, Python 3.10.14). What I'm confused about is the CI/CD failure at https://github.com/move-coop/parsons/actions/runs/8869754280/job/24350778446?pr=1045

anzelpwj commented 2 months ago

I will note that https://github.com/move-coop/parsons/pull/1044 also failed at the same step and in the same way.