🇫🇷France @S3b0uN3t

No response, I close the issue.

@kgaut, don't hesitate to come back to us if your problem persists.

Hi @kgaut,

Are you still making this bug?
If so, can you provide more information?

Without feedback, I will close the issue.

For a simple change I prupose to add these lines in css/ckeditor.css :

/* Manage "Gin" theme dark mode. */
.gin--dark-mode .ck-content {
  --grey-200-850: #cecece;
  --grey-200-850-hover: #a8a8a8;
  --grey-200-850-active: #939393;
}

Extracted from DSFR library dark mode :

• File dist/scheme/scheme.css
• Selector :root[data-fr-theme=dark]

I think the best approach would be to use CkEditor 5's CSS variables as much as possible: https://ckeditor.com/docs/ckeditor5/latest/getting-started/setup/css.html#customizing-the-editors-look.

Example for the lark theme (used for dark mode): https://ckeditor.com/docs/ckeditor5/latest/framework/deep-dive/ui/theme-customization.html#customization-with-css-variables

Hi @kgaut,

Color management in dark mode is actually not interpreted correctly.
Your scenario will work well with Gin but the problem may arise for other dark themes.

Also, are we sure that it is only necessary to declare this property for everything to work correctly?

Hello,

Thanks for the detail of your issue.

My first tests do not demonstrate the problem on the sites where the module is implemented.
This bug is probably be related to Captcheat official assets or your website but not the module implementation.

I will carry out further testing as soon as I have availability.

Thanks for your review.

I close the issue.

I've checked with original healtcheck API :

In sandbox mode (with sandbox credentials) : https://sandbox-api.piste.gouv.fr/piste/captcha/healthcheck`

array(2) {
  ["status"]=>
  string(2) "UP"
  ["components"]=>
  array(4) {
    ["captchaTestHealth"]=>
    array(1) {
      ["status"]=>
      string(2) "UP"
    }
    ["diskSpace"]=>
    array(2) {
      ["status"]=>
      string(2) "UP"
      ["details"]=>
      array(4) {
        ["total"]=>
        int(0)
        ["free"]=>
        int(0)
        ["threshold"]=>
        int(0)
        ["exists"]=>
        bool(true)
      }
    }
    ["memcachedHealth"]=>
    array(1) {
      ["status"]=>
      string(2) "UP"
    }
    ["ping"]=>
    array(1) {
      ["status"]=>
      string(2) "UP"
    }
  }
}

And in production mode (with production credentials) : `https://sandbox-api.piste.gouv.fr/piste/captcha/healthcheck` :

array(2) {
  ["status"]=>
  string(2) "UP"
  ["components"]=>
  array(4) {
    ["captchaTestHealth"]=>
    array(1) {
      ["status"]=>
      string(2) "UP"
    }
    ["diskSpace"]=>
    array(2) {
      ["status"]=>
      string(2) "UP"
      ["details"]=>
      array(4) {
        ["total"]=>
        int(0)
        ["free"]=>
        int(0)
        ["threshold"]=>
        int(0)
        ["exists"]=>
        bool(true)
      }
    }
    ["memcachedHealth"]=>
    array(1) {
      ["status"]=>
      string(2) "UP"
    }
    ["ping"]=>
    array(1) {
      ["status"]=>
      string(2) "UP"
    }
  }
}

At the launch of V2 (in June 2024), not all APIs were available in "/v2". This apparently is now the case. I plan to migrate them all to the new version.

My patch for fix site install on 3.x version.

I get the same error on site install with existing configurations in drush command.

Hi @kgaut,

I took the liberty of retouching your Merge Request.
This bug only occurs if the "Navigation" module is activated, I prefer that we overload the header in this case.

If you can validate what I did. ;-)

Hello,

This module is used in several projects without problem today.

I consider your issue and try to check it (and your patch) as soon as possible.

Hello !

I've tested your changes. It's ok for installing project but an error occurs on config form :
ArgumentCountError: Too few arguments to function Drupal\Core\Form\ConfigFormBase::__construct(), 1 passed in /var/www/web/modules/contrib/domain_simple_sitemap/src/Form/DomainSimpleSitemapConfigForm.php on line 73 and exactly 2 expected in Drupal\Core\Form\ConfigFormBase->__construct() (line 43 of core/lib/Drupal/Core/Form/ConfigFormBase.php).

It seems to be a Drupal 11 compatibility error.

Hi @kgaut !

Thanks for your feedback and your Merge Request. I check it as soon as possible.

Hi,

The submitContactForm() method is called both during postSave() and during validation if the form contains a phone field.
This method ends with the creation of a contact on Brevo, which should normally only happen at postSave().
I propose a workaround so as to no longer create contact upon validation.
This is not ideal; In my opinion, the design of this part of the module will have to be reworked.

Cheers,

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

s3b0un3t → created an issue.

Hello,

I meet the same message with "paragraphs_library_item" entity type on URL "/devel/token/paragraphs_library_item/{id}": "Exception: Object of type Drupal\paragraphs\Entity\ParagraphsType cannot be printed".

It try to generate a token "[paragraphs_library_item:paragraphs:entity:type:entity]" as EntityType object.

Your patch seems to fix my issue. It's not perfect but the page loaded correctly.

Thanks !

Hello,

We have just generated new releases to manageDrupal 11 compatibility with SDC module :

• A Drupal 1.1.2 release with Drupal 11 compatibility removed (based on new protected branch 1.1.x)
• A Drupal 1.2.0 release with SDC dependency removed (base on default branch 1.x)

Thanks for your help.

Cheers,

Hello @kgaut,

Thanks for the additional documentation.

I gave you a brief feedback in the Merge Request.

Cheers,

HI kgaut,

Thanks for your feedback and your merge request!

We check it soon.

Cheers,

Hello Kevin,

The dependency is directly made on the SDC module of the Drupal code (drupal:sdc) and not the community module (sdc:sdc).

So I don't understand why you can't install the theme with Drupal +10.1 code.

It is actually not a good idea to remove this dependency, otherwise the inclusion of components would no longer work on a site without the SDC module.

Even if the SDC module no longer does anything in Drupal 11, it is still present: https://git.drupalcode.org/project/drupal/-/blob/11.x/core/modules/sdc/s.... yml?ref_type=heads

I will look into this problem as soon as I have some time (I am in the middle of house moving).

We can discuss this topic via Slack if you wish.

Hello Kevin,

The "navbar" variable is by default defined with the content of the "header_navbar" region in the "page.html.twig" and "page--404.html.twig" templates.

If you have declared a child theme, make sure that this region exists in your new theme.

In an evolution, we can also consider setting a default value if the region does not exist.

You can contact me via Slack if necessary.

Hello,

There is currently no stable version of this module. This is a draft which is only available in development version.
The choice to use it is at your own risk.

In my last tests, all examples were functional.
Have you looked at the logs to identify any errors reported by the DocuSign API response?

If you ever identify a bug, don't hesitate to suggest a patch or a merge request.

Sincerely,

Helllo,

Thanks for your feednack. I have fixed all missing returns.

Cheers,

Hello,

I have just generated a 1.3.0 release which ensures compatibility with the Address module in version 2.x.
This major version of the Address module does not cause incompatibility with previous versions of my module.
That's why, release 1.3.0 still remains compatible with versions 1.x of the Address module.

Cheers,

Hello,

Thanks for your feedback.

I try to purpose a new version with address >=2.0 compatibility as soon as possible (at best this weekend).

Cheers

Hello !

I applied your patch and created a new release.

Thanks for your help ! :)

Hello.

You’re all right. I’m currently in holidays without computer. I apply your patch as soon as possible.

Thanks

S3b0uN3t → created an issue.

It's done !

Don't hesitate to let me know your opinion. ;-)

Hello,

I will try to find time next weekend to provide an example sub-module and a screen capture of it.

Fixed in main branch

Fixed in main branch

Fixed in main branch

Fixed in main branch.

Fixed in main branch.

Fixed in main branch.

S3b0uN3t → changed the visibility of the branch 1.0.x to hidden.

Hello,

Thank you for your task. I totally agree.

You have not created a merge request.
I commit the modification to the project directly
I generate a new release immediately.

Hello,

Thank you to all three for this feedback, the patch and the tests.

I tested it on my side and everything seems ok.

I apply the patch and generate a new release.

Tested with version 3.8 of the AMP module in PHP 8.3.
The Merge Request resolves the deprecated issue and does not result in a new issue.

I update the issue in RTBC.

Thanks for your fix !

Hello,

I tried to apply patch on comment #50, but it's thrown an error in datetime module :

Warning: Undefined array key "type" in Drupal\datetime\Plugin\views\filter\Date->getOffset() (line 180 of core/modules/datetime/src/Plugin/views/filter/Date.php).

Without this warning, the patch works perfectly on Drupal 10.2.2!

Hello,

@camilo.escobar : I've try to use your patch in issue #2982968 but it's return an error.
I will add a comment on your issue in order to trace the error.

The patch #25 seems to work on Drupal 10.2.2.

Cheers.

Hello,
It seems that this function was accidentally deleted.
It ensures that the tokens used in custom parameters do not infringe the GDPR (prohibition on the transmission of private data).

I added this method.
It is based on the same code as the Matomo module.

I am pushing a fix and will generate a new release quickly.

THANKS !

Hello,

When submitting the form, you need to generate a DocuSign envelope and redirect user to DocuSign (as in my examples).

In a Webform context, you need to use your own WebformHandler → to add you redirection in ::postSave() (Webform example here) in which you will do the same redirection principle as in my examples.

Hope my answer helped you.

Cheers,

Hello,

The module is currently available in a development version and cannot be considered stable. Its use is at your own risk.

However, I would like to help wherever possible.

I confirm that the electronic signature examples are now operational, provided that the template and associated fields exist

Simply follow the DocuSign example and documentation if you want to perform more advanced operations.

Can you provide a clear example of the workflow you are trying to do ?

S3b0uN3t → created an issue.

Hello,

I have just created a 1.1.0 release which implements a "font_iconpicker" form element.
It is now possible to use this widget in custom forms.

Hello,

The module does not currently allow the "IconPicker" field to be used as a simple form element. It is dependent on Field UI.

However, it is possible to consider its implementation.
If you have the skills, don't hesitate to submit a Merge Request, otherwise, I will try to find the time to set it up soon.

Hello,

I moved the access token check to the route rather than the form. This no longer seems to produce redirection problems when caches are activated (anonymous as well as connected).

Hello,

Thank you for the clarification.
I now understand the origin of the problem and will come back to you in the next few days with a fix.

Good morning,

The status error is often linked to expired transaction issues.

You reported a cache problem to me in issue 3382502 🐛 Website getting issue on redirecting on Oauth callback. Fixed which has since been corrected.

It is impossible for me to debug your query with this error without context. I updated the exception so that it returns the current state and the expected state to help you to investigate.

Hello,

Changes are already available in dev version.

The « invalid grant » error provides when you call Docusign OAuth callback url without query parameters or if you don’t have a state define in temporary store (this state is defined on each Docusign OAuth request).

By default, the callback is the current page (the page where you request Docusign). Don’t hesitate to create a task and purpose a merge request to define a default callback path.

Cheers

Hi,

After several tests, it turns out that the OAuth callback is operational (both with Code Grant or JWT Grant).

However, I noticed caching issues on the OAuth callback when page caching was enabled.
I fixed this problem.
In the context of normal use, we are now redirected to the initial page (and no longer to "/docusign/signature/callback/oauth" without a parameter in the URL).

Good luck for the future and don't hesitate to make suggestions for developments (or submit them).

Thanks for your proposal. I commit changes.

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

I won't have time to test until next week.
Feel free to provide a fix as a path or merge request.

Sorry, I didn't see your comment yesterday.
Our comments crossed.

Can you tell me more about the situation in which you encounter this error?
The GET parameter must correspond to what is normally declared in session.

Can you check what this GET parameter is worth and what you have in session when it throws you the error?

Hello,

You are absolutely right. It's a misprint on my part.

Thank you for your feedback and your attention! :)

The proposed patch now only applies to attributes using single quotes. I'm updating the patch (for 9.5.x) to work for double quotes as well.

I committed a change on the access to the route. Don't hesitate to tell me if you don't like it.

The permission "use text format advanced" is available when a text format config entity "advanced" is available.
I think create a custom permission with "use text format advanced" syntax is not a good idea.

You can create a custom permission like "use ckeditor templates" if you want to manage a special access to the ckeditor button.

Any response since 10 months. I close the issue.

No response for over a year. I close the issue.

No return for one year. I close the issue.

No return for one year. I close the issue.

Hello,

Javascript files should not be parsed with PHPCS as they do not check the same rules.
They must follow the AirBnB rules as designated in the "core/.eslintrc.json" file.

Hello,

This mode of operation also seems more logical to me and allows you to resume the theme without "Vite" if necessary (in the context of reversibility for example).

As the operation differs, it may be interesting to create a new branch (2.x?) in order not to break the sites currently using the 1.x version.

I tested what was done by @Hugo_C → . I change the status of the ticket.

Thanks for your proposal. I commit changes.

It's good for me. Thanks for your help !

Thanks for your feedback and your patch.

I apply it and fix other warnings (if there are still) as soon as possible and generate a new dev release.

S3b0uN3t → created an issue.

Thanks for your proposal. I commit changes.

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

Issue fixed in new release 1.1.2

Hi

Thanks for your return. I make a fix and generate a new release as soon as possible.

Add @Mir4 purposed patch in comment #114.