feat/google drive

[midepeter]

I made this initial push for the google drive feature.

[opensaucerer]

Thank you very much for this @midepeter
However, I noticed you might not have checked the contribution guide yet.
Please check it out for proper flow on how we hope to streamline the development of new features or fixes.

[midepeter]

#9

[opensaucerer]

@midepeter My sincerest apologies for the delay on the review here....

Some breaking changes have been made to Bifrost recently and might affect your current PR.

Will you be able to update the PR so I can review again or should I make the updates for you? 🙇

[midepeter]

@opensaucerer hello, alright I will look into this.

[opensaucerer]

@opensaucerer hello, alright I will look into this.

Thank you 🙇

[opensaucerer]

Hi @midepeter
Here's an update on the recent changes to bifrost's codebase. The interfaces have been updated and functions to UploadFile have been modified to take structs as input instead of an enumeration of arguments.

If you pull the latest changes, you'll notice these refactors.

If you will need help refactoring this PR, do let me know. Thank you for contributing.

[midepeter]

Alright noted

[midepeter]

@opensaucerer i really do apologise that i have not been so consistent with this issue but i will surely complete it.

are you going to review what i have done so far? becfore i go ahead and complete it and then test.

[opensaucerer]

Hi @midepeter
It's completely fine, I very much understand.
I also went away for 2 months on the project, so much work at hand.

Yeah, I reviewed the work but didn't comment because I hoped to wait on the refactor to fit the new bifrost structure.

But I can leave comments if you do want me to do that now.

[midepeter]

@opensaucerer

Yes, you can leave the comments already and i will work on it. Or if you want me to refactor first then no problem

[opensaucerer]

UseAsync is missing and google drive doesn't use DefaultBucket

[midepeter]

@opensaucerer

I do not understand this comment. If you could help shed more light on it a bit

[opensaucerer]

Hi, sure.

One of the configurations that should be returned is UseAsync and one of the configurations currently being returned in this provider includes DefaultBucket which is not needed.

[opensaucerer]

@midepeter I've added the comments. Thank you.