uno icon indicating copy to clipboard operation
uno copied to clipboard

StorageFileHelper - Add GetFilesInDirectoryAsync

Open rafael-rosa-knowcode opened this issue 1 year ago • 6 comments

GitHub Issue (If applicable): closes #

PR Type

What kind of change does this PR introduce?

  • Feature

What is the current behavior?

The StorageFileHelper just have one method for Determines if an asset or resource exists within application package. But could be a nice feature to retrieves the paths of files within a directory.

What is the new behavior?

So this PR is for allow to retrieves the paths of files within a directory based on a specified filter for file extensions (If null, all files in the directory are considered.) And the result is an array of strings containing the paths of the filtered files within the directory.

PR Checklist

Please check if your PR fulfills the following requirements:

Other information

Internal Issue (If applicable):

rafael-rosa-knowcode avatar Apr 17 '24 19:04 rafael-rosa-knowcode

@nickrandolph Can you confirm if you agree with the code? @jeromelaban Can you take a look and comment or direct someone?

rafael-rosa-knowcode avatar Apr 17 '24 19:04 rafael-rosa-knowcode

🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-16341/index.html

unodevops avatar Apr 17 '24 19:04 unodevops

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-16341/index.html

unodevops avatar Apr 17 '24 19:04 unodevops

@jeromelaban one of the questions we had here is whether we can make StorageFileHelper public. We need to access it, so we can either make it public or internal, with internals access granted to the appropriate library. Thoughts?

nickrandolph avatar Apr 18 '24 03:04 nickrandolph

@jeromelaban one of the questions we had here is whether we can make StorageFileHelper public. We need to access it, so we can either make it public or internal, with internals access granted to the appropriate library. Thoughts?

I think public is preferable even if it is not intended for usage by users, as we can then more easily avoid making unintended breaking changes when we were to adjust naming some other first-party library depends on for example

MartinZikmund avatar Apr 18 '24 08:04 MartinZikmund

@jeromelaban one of the questions we had here is whether we can make StorageFileHelper public. We need to access it, so we can either make it public or internal, with internals access granted to the appropriate library. Thoughts?

What do you need from storage helper?

jeromelaban avatar Apr 23 '24 15:04 jeromelaban