Delete Dimension

[ahmedrizwan239]

Implementation of the deleteDimension api call.

[theoephraim]

nice! Thanks :D

To make the DX slightly easier, I think I might

• make 2 methods deleteRows and deleteColumns - this could be in addition to deleteDimension, and could just call deleteDimension internally
• unwrap the start/end params from the object (ie deleteRows(start, end)) but be sure to handle the undefined case (google's api lets you have it unbounded on one side)

It's all fairly subjective, but it's sort of the fun part in trying to design a more intuitive interface for users than google's apis, which is what this module is all about.

Also we'll need to add tests and docs before merging.
Feel free to continue, or leave it as is and I'll add the rest before merging.

Cheers and thanks!

[ahmedrizwan239]

Can you please merge this asap? I need to use it in my package for a project. This would help, Thanks!

[cesar3030]

Agree, could this be merged please?

[fernandohg97]

This will work great in my project! Could this be merged please?

[TRMoosee]

Could definitely use this on a project I'm working on, is there any way we could get this merged soon?

[ururk]

Could use this as well!

[ururk]

@theoephraim It looks like some changes you suggested still need to be done... should I fork @ahmedrizwan239 fork or make my own?

[theoephraim]

@ururk - yeah if you're up for it, got for it! The functionality itself is pretty simple, but need to add tests, docs, and ideally make the DX a little simpler... Otherwise I'll try to get to it asap

[ururk]

I'm taking stab at it, hopefully have something for today. I have the two additional methods written and was looking at your tests

[theoephraim]

closing in favour of #745 - sorry for the delay and thanks!