Account created on 29 March 2012, about 13 years ago
#

Merge Requests

More

Recent comments

🇳🇱Netherlands idebr

The merge requests fixes the phpunit tests

🇳🇱Netherlands idebr

idebr created an issue.

🇳🇱Netherlands idebr

In the merge request, the module claims it is compatible with Drupal Commerce 3. This means the changes from 📌 Commerce 3.0 campatibility Active are required for compatibility.

In addition the MollieApiMock determines the payment status from the paid amount using incorrect comparisons, eg. 29.99 === 29.9900000. I updated MollieApiMock to compare Prices instead.

The functional tests are now green.

🇳🇱Netherlands idebr

idebr made their first commit to this issue’s fork.

🇳🇱Netherlands idebr

My project requirement is pretty basic:
- Upload a file in a node
- Automatically translate the file when translating the node using Deepl

This is supported by tmgmt, see Translate files / attachments Fixed

The priority for automatic translations for documents in my project has been lowered, so I don't expect to do any development on this feature in the short term

🇳🇱Netherlands idebr

\Drupal:state() is now replaced with \Drupal::keyvalue('user_form_test') in the test coverage

🇳🇱Netherlands idebr

The merge request fixes the remaining phpstan issue

🇳🇱Netherlands idebr

The patch in #2 is now available as a merge request

🇳🇱Netherlands idebr

idebr made their first commit to this issue’s fork.

🇳🇱Netherlands idebr

The merge request does not lock the entity when adding a content translation

🇳🇱Netherlands idebr

The merge request removes the obsolete drupalci.yml file from the project.

🇳🇱Netherlands idebr

Fixed the last test failure.

🇳🇱Netherlands idebr

This is being fixed in 🐛 Undefined function field_ui_entity_operation Active . I'll close this issue as a duplicate, so we can focus our efforts in the related issue.

🇳🇱Netherlands idebr

This is being fixed in 🐛 Undefined function field_ui_entity_operation Active . I'll close this issue as a duplicate, so we can focus our efforts in the related issue.

🇳🇱Netherlands idebr

The item category contains additional logic to add a URL attribute when it uses a term reference field.

Using a field that is not a term reference will now no longer trigger a PHP exception

🇳🇱Netherlands idebr

The merge request fixes the phpcs pipeline job

🇳🇱Netherlands idebr

idebr made their first commit to this issue’s fork.

🇳🇱Netherlands idebr

I filed an issue to fix 1.x as well while it is still supported, see 🐛 [1.x] Fix phpstan and commerce plugin contructors Active

🇳🇱Netherlands idebr

.ck-editor includes the toolbar. Let's use .ck-content so the css only applies to the editable content.

🇳🇱Netherlands idebr

The issue summary is updated with the issue summary template heading.

🇳🇱Netherlands idebr

Can you check the Grant type 'Refresh Token' is enable on your Consumer edit form?

🇳🇱Netherlands idebr

The merge request adds 'roles' to AccountForm getEditedFieldNames() and flagViolations(), so Constraint violations are properly displayed as form errors.

🇳🇱Netherlands idebr

Yes, we should be explicit about ^9.0 of league/oauth2-server due to PHP compatibility of method signatures

🇳🇱Netherlands idebr

@duaelfr can you update the merge request target to branch 3.0.x? This option is only available for the creator (or maintainers)

🇳🇱Netherlands idebr

idebr made their first commit to this issue’s fork.

🇳🇱Netherlands idebr

@dewalt can you tag a new release, so this module can be installed with Drupal 11?

🇳🇱Netherlands idebr

Rerolled against the latest 6.0.x

🇳🇱Netherlands idebr

The merge request fixes the PHP warnings when a view has multiple Facet summaries

🇳🇱Netherlands idebr

The merge request changes the button text to use the default when adding a content translation for a group.

🇳🇱Netherlands idebr

The dependency is implicit since the config file name starts with rabbit_hole

🇳🇱Netherlands idebr
  1. simple_oauth_cron() now checks for falsy values for token_cron_batch_size and defaults to 0
  2. token_cron_batch_size is now a required field on the config form, since it is not nullable
  3. Added the 0 limit to the description of the form item on the config form and schema
🇳🇱Netherlands idebr

The table class was updated from sticky-enabled to sticky-header in Drupal 10.3, see https://www.drupal.org/node/3440477

🇳🇱Netherlands idebr

The merge requests implements the following changes:

  1. The 'Issued by' (iss) claim is now generated without a language prefix
  2. The OpenIdConnectTest now checks for the correct URL, see https://git.drupalcode.org/issue/simple_oauth-3502576/-/jobs/4153025
🇳🇱Netherlands idebr

The z-indexes in Drupal are documented and considered stable.

I'm unable to reproduce this issue on 11.x, .ui-widget-overlay gets a z-index of 1259 and is displayed over the sticky header

🇳🇱Netherlands idebr

@eojthebrave Is it possible to tag a new release, so downstream modules that require a stable release for Consumers have a green pipeline again? See for example https://git.drupalcode.org/project/simple_oauth/-/jobs/3848264

🇳🇱Netherlands idebr

The test checks for the aria-describedby attribute. This attribute is only available when the field has a description.

🇳🇱Netherlands idebr

The remaining deprecations are now fixed in the merge request

🇳🇱Netherlands idebr

idebr made their first commit to this issue’s fork.

🇳🇱Netherlands idebr

The merge request updates configuration forms to use #config_target

🇳🇱Netherlands idebr

The merge request raises the minimum supported Drupal version to 10.3: https://endoflife.date/drupal

The logger can now be autoconfigured, see https://www.drupal.org/node/3395436

🇳🇱Netherlands idebr

I have created a merge request to either fix or disable cspell. Only one should be merged.

PHPUnit failures are caused upstream, see 📌 PHP 8.4: Implicitly nullable parameter declarations deprecated Active

🇳🇱Netherlands idebr

Considering Drupal 11 has been released a while ago, this issue can now be marked as fixed?

🇳🇱Netherlands idebr

This issue also happens to be caught by phpcs, so I fixed these issues in a single go

🇳🇱Netherlands idebr

The merge request removes lingering 'priority' keys from node access records

🇳🇱Netherlands idebr

🌱 Move to thephpleague/oauth2-server 9.0 Active has been committed. However, there is no upstream support yet for introspection

Production build 0.71.5 2024