gitlabr icon indicating copy to clipboard operation
gitlabr copied to clipboard

File operations with folders in path

Open mpolano opened this issue 3 years ago • 1 comments

Hi Sébastien,

I hope I've correctly interpreted your intention for the URL encoding. Now all file functions accept a file_path that includes folders like "test-folder/dataset.csv" and each function takes care of URL encoding appropriately.

In order to make it work I've also modified gl_file_list() adding a parameter to specify the path. This requires on the API call to add a query parameter (as documented here).

I found what I think is a bug in gl_file_exists() that I've fixed: looking for files in an empty repository produced an error.

I have added tests to take into account folders trying to follow what you already did without folders.

mpolano avatar Aug 25 '22 16:08 mpolano

Thanks. I'll test it in a few days, when back from holidays :smiley:

statnmap avatar Aug 25 '22 22:08 statnmap

Less than 2 years to achieve this one...
This was a big one, and I am happy to close it :) Thank you for your nice contribution!
I retrieved all your commit in this PR: https://github.com/ThinkR-open/gitlabr/pull/106

statnmap avatar May 14 '24 14:05 statnmap

It's never too late 🙂. I'm glad to have contributed to this project.

Keep up the good work !

On Tue, 14 May 2024 at 16:58, Sébastien Rochette @.***> wrote:

Less than 2 years to achieve this one... This was a big one, and I am happy to close it :) Thank you for your nice contribution! I retrieved all your commit in this PR: #106 https://github.com/ThinkR-open/gitlabr/pull/106

— Reply to this email directly, view it on GitHub https://github.com/ThinkR-open/gitlabr/pull/74#issuecomment-2110466326, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEGGYN5IMLCQZORB35HPHDTZCIRCLAVCNFSM57TX2FJ2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJRGA2DMNRTGI3A . You are receiving this because you authored the thread.Message ID: @.***>

mpolano avatar May 17 '24 13:05 mpolano