Vincenzo Mecca

Results 17 comments of Vincenzo Mecca

@tdonohue created a PR #1816 for this ISSUE, now is in waiting for reviewers.

Hello @tdonohue, now all change requests should be resolved. Thank you.

Hello @tdonohue, now all change requests should have been resolved. Thank you.

Thank you, @tdonohue. > All in all, I think this is good to merge. We'll need to update the documentation about how to enable/configure Amazon S3 storage. I suspect we...

@abollini @tdonohue @pnbecker PR updated with requested changes. Thank you!

Hello @tdonohue , thank you for the review, now it should work.

@tdonohue @abollini : all requested changes are now resolved. Thank you.

> @vins01-4science : I re-reviewed this today and the code looks good. I manually called each of the new endpoints, and they all work **except for `listhandles`** (which always returns...

Hello @sheilarabun @tdonohue , We @4Science can work on this issue. I'm going to self-assign this one. Thank you.

Hello @tdonohue, We @4Science can work on this issue if you agree on the proposed fix. Thank you!