πŸ‡©πŸ‡ͺGermany @Erik Seifert

Account created on 9 July 2007, over 17 years ago
  • Drupal Solution Architect at 1xINTERNETΒ  …
#

Merge Requests

Recent comments

πŸ‡©πŸ‡ͺGermany Erik Seifert

Should you not get the credit with the commit ?

πŸ‡©πŸ‡ͺGermany Erik Seifert

Hard to achieve. If somebody has a resolution feel free to post it here.

πŸ‡©πŸ‡ͺGermany Erik Seifert

Having a log message would be a good addition. The file should be there, so it's warning.

πŸ‡©πŸ‡ͺGermany Erik Seifert

I would replace the current js implementation. It seems more comfortable and modern.

πŸ‡©πŸ‡ͺGermany Erik Seifert

We should have a better label than "Include destination". Something like "Return to dashboard after create content".

πŸ‡©πŸ‡ͺGermany Erik Seifert

Thank you for support and work on this module. And after your patch is looking very nice. I like it ;-) Seems your MR does not have any changes, so i commit the patch.

πŸ‡©πŸ‡ͺGermany Erik Seifert

Erik Seifert β†’ made their first commit to this issue’s fork.

πŸ‡©πŸ‡ͺGermany Erik Seifert

Yes it is a colleague from we. Works about 10 years with drupal projects.

πŸ‡©πŸ‡ͺGermany Erik Seifert

@hchonov. This setting is handled by klaro itself. The modal is not the second screen. There is also a cookie banner, but that's not the modal.

πŸ‡©πŸ‡ͺGermany Erik Seifert

Looks for me that is works like designed.

See first screenshot shows only the small cookie banner in the bottom.
Second shows the modal without any possibility to close.

Seems for me the correct behavior.

πŸ‡©πŸ‡ͺGermany Erik Seifert

- Added eslint config to have an easy check
- Utilize once in klaro.js

πŸ‡©πŸ‡ͺGermany Erik Seifert

@elber feel free to test.

πŸ‡©πŸ‡ͺGermany Erik Seifert

I have some findings with the tests.

- Added none initiated plugin manager
- Add correct behavior for empty exclude_paths
- Better block rendering (Bypass if user has the permission)

πŸ‡©πŸ‡ͺGermany Erik Seifert

The current schema files does not apply any schema on simple_klaro.settings. Update hook ist not needed here.

πŸ‡©πŸ‡ͺGermany Erik Seifert

See πŸ› Config schema incorrect name Closed: duplicate

Production build 0.71.5 2024