Michael Jansen

Results 209 comments of Michael Jansen

Mantis Issue is: https://mantis.ilias.de/view.php?id=41574

Mantis Issue is: https://mantis.ilias.de/view.php?id=41574 I am personally not convinced that this is the best solution, so IMO this should not be fixed this way. Instead a proper DIC should be...

Hi @chfsx , we stumbled over this PR while checking old PRs in our TB meeting. Please decide if you want to close the PR, request changes, request a new...

Hi @jcopado , just one thought on this: If you want to make these dependencies explicit (which is good, thanks a lot) by adding them to the `composer.json` file, you...

Hi @smeyer-ilias , we stumbled over this PR while checking old PRs in our TB meeting. Do you have any objections (maybe you reject the PR completely)? Providing feedback for...

Thx for providing more examples @nhaagen . Since this PR targets the UI component(s), I change the assignment.

Hi @qualitus-dahme , I added your account to the list of assignees. Please keep in mind that this fix needs to be picked to all maintained branches (`release_9`, release_10` and...

Hi @pascalseeland , a detailed/profound review and feedback would be appreciated here. Best regards, Michael

Hi @pascalseeland, a review is still highly appreciated. Best regards, Michael