Set requirement on file for main module

Hi Tim,

I will check this out.

Greetings,

Fabio

abhiyanshu → made their first commit to this issue’s fork.

@tim-diels,
I have set the requirement in the module for the file. Please refer to the screenshot for more clarity. Thanks.

Hi abhiyanshu,

Thanks for doing the work that needed to be done, only I did already mention I was going to take this up.

In the future, it is best that you first check if someone else has not already specified that they are working on this before starting the ticket.

This also ensures no duplication of work, which is certainly important for larger tickets.

It is also best practice to transfer the ticket to "Needs review" after the work has been done, so other developers know it needs to be looked at.

I will test your solution.

Greetings,

Fabio

Hi Tim,

I tried the code change and the site now asks to turn on file and I no longer get a fatal error.

Greetings,

Fabio

@dtfabio, I waited for a while since the changes were not that big, so I went ahead and pushed them.
I will make sure to follow up on those things next time. Thank you for your valuable suggestions and the review. I appreciate your guidance!

Hi abhiyanshu,

To be frankly, you waited for 26 minutes. So you don't give a chance for someone to set up the project and do the code changes ...
Thats not very friendly from your side. But as we're a friendly community and I was working together with Fabio to pick this up, I gave the advice to make you aware of it. I don't think your response is that professional so maybe something to work on next time you work on issues.

Thank you both for the work and tests. I'll get this comitted.

Seems we fixed the dependency in another issue 📌 Transform the architecture to use a dedicated streamwrapper Fixed so don't need this anymore.