delete_row(idx) about gspread HOT 9 CLOSED

I think we need to give him some more time. I mean, it's only been like, four and a half years.

from gspread.

Alright, I created a fork and built on amleczko's work. I added the ability to delete by row numbers, delete a list of rows without it failing silently and wrote some more tests.

I've submitted a PR. It's passing all tests and there's no merge conflicts. #390

from gspread.

delete_row() was added in #337, v0.5.0. I will add code from #161 and #390 in future releases. I'm closing this issue.

from gspread.

Thank you, Peter, I'm sure we need it in the library!

from gspread.

+1 for delete_row(), would come in very handy!

from gspread.

Hi burnash

I too would like delete_row().

I did take a look at the implementation but I can't find any documentation for the google API at the http level (I can only find their API library documentation, which does have a delete_row function).

Where have you been to find the API specifics, did you reverse engineer it all or is there a reference somewhere that I can read?

Thanks for the library!

Richard

from gspread.

Bump. We should add delete_row.

from gspread.

Is this ever going to happen?

from gspread.

@NotLeet you could always help to make it happen faster. The referenced PR misses a test suite and for now I don't have enough time to write it myself. I'm sure that not every gspread user will be happy if I merge all incoming PRs without any prior testing.

This is a non-commercial, open source project. If you use gspread and need something — please contribute to the project.

from gspread.