update required version of teal.logger

[chlebowa]

fixes #1348

[donyunardi]

Thanks @chlebowa !

[vedhav]

Before we merge I just wanted to confirm if this is needed?
The teal.logger::log_shiny_input_changes is only used in module packages and teal does not use them.
The issue mentioned in this PR originates from the tmg and we already have the min version of teal.logger (>= 0.2.0.9004) in tmg which has the function log_shiny_input_changes exported.

[chlebowa]

Fair point @vedhav, I didn't test with example_module(). Perhaps this fix belongs in module packages.

[donyunardi]

Thanks @vedhav I think you're right.

The related PR in question is:
insightsengineering/teal.modules.general#760

@chlebowa can you confirm on your end?

[chlebowa]

Now that I have the latest teal.logger installed, it doesn't matter which package forced it.
However, I'm afraid I will run into the same issue the next time I want to use tmc.

[donyunardi]

However, I'm afraid I will run into the same issue the next time I want to use tmc.

In tmc we also forced the teal.logger to be >= 0.2.0.9004
https://github.com/insightsengineering/teal.modules.clinical/blob/1d4beec4ef14197445191b0957bb38e8bba9196d/DESCRIPTION#L60

Are you also using the latest tmc (from main) when you get the error message?

[chlebowa]

At the moment I am not getting the error emssage even using tmc v0.9.1 but, again, my teal.logger is already updated.

Sorry, I should have been more specific. If I had not updated teal.logger just now (I was not forced to do so by teal or any other package), I would get the same error. And if the issue is remedied in tmg only, tmc will be vulnerable.

[donyunardi]

At the moment I am not getting the error emssage even using tmc v0.9.1 but, again, my teal.logger is already updated.

This makes sense because the teal.logger::log_shiny_input_changes in tmc was implemented after v0.9.1.

However, if you install tmc from main branch and you only have teal.logger 0.2.0, teal.logger would have been updated (or prompted for an update) since tmc requires teal.logger to be >= 0.2.0.9004.

Regardless, I will close this PR without merging since we already force this when using the teal modules.