Skip to content

Conversation

@LucianDeveloper
Copy link

No description provided.

@LucianDeveloper LucianDeveloper changed the title fix-147 fix-147 - Fix getting file for storage name with complex path Apr 23, 2024
Copy link
Owner

@jowilf jowilf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix. Can we add a test case to ensure this won't break again in the future?

@NickNaskida
Copy link
Contributor

NickNaskida commented May 15, 2024

@LucianDeveloper Hey, any updates regarding this? Can we get this thing merged?

@NickNaskida
Copy link
Contributor

@jowilf Can we somehow get this merged? If not how can I help to merge this in? Should I create a new pull request with this feature and test case?

@jowilf
Copy link
Owner

jowilf commented Jul 2, 2024

@NickNaskida, we need a simple test case. I haven't had the time to work on it myself yet. The quickest way to merge it is to create another PR with a test case, as you can't commit to this branch directly.

@NickNaskida
Copy link
Contributor

Just did that, please take a look once you have time. Thanks!

@jowilf
Copy link
Owner

jowilf commented Sep 8, 2024

this is now implemented in #154

@jowilf jowilf closed this Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants