The merge request does not lock the entity when adding a content translation
idebr → created an issue.
The merge request removes the obsolete drupalci.yml file from the project.
Fixed the last test failure.
A non-translatable string should just be 'string', see for reference: https://www.drupal.org/docs/drupal-apis/configuration-api/configuration-... →
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.
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.
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
Can now be 10.3 as 10.2 is end-of-life
📌 Test with drupal/video_embed_field 3.x for Drupal 11 compatibility Active has been committed.
Committed to allow for further testing.
idebr → created an issue. See original summary → .
The merge request fixes the phpcs pipeline job
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
.ck-editor includes the toolbar. Let's use .ck-content so the css only applies to the editable content.
The issue summary is updated with the issue summary template heading.
Can you check the Grant type 'Refresh Token' is enable on your Consumer edit form?
The merge request adds 'roles' to AccountForm getEditedFieldNames() and flagViolations(), so Constraint violations are properly displayed as form errors.
Confirmed the issue still occurs in 11.x
Yes, we should be explicit about ^9.0 of league/oauth2-server due to PHP compatibility of method signatures
@duaelfr can you update the merge request target to branch 3.0.x? This option is only available for the creator (or maintainers)
@dewalt can you tag a new release, so this module can be installed with Drupal 11?
Rerolled against the latest 6.0.x
The merge request fixes the PHP warnings when a view has multiple Facet summaries
The merge request changes the button text to use the default when adding a content translation for a group.
The dependency is implicit since the config file name starts with rabbit_hole
idebr → created an issue.
- simple_oauth_cron() now checks for falsy values for token_cron_batch_size and defaults to 0
- token_cron_batch_size is now a required field on the config form, since it is not nullable
- Added the 0 limit to the description of the form item on the config form and schema
The table class was updated from sticky-enabled to sticky-header in Drupal 10.3, see
https://www.drupal.org/node/3440477 →
The merge requests implements the following changes:
- The 'Issued by' (iss) claim is now generated without a language prefix
- The OpenIdConnectTest now checks for the correct URL, see https://git.drupalcode.org/issue/simple_oauth-3502576/-/jobs/4153025
The merge request updates the dataProvider to be a static method.
idebr → created an issue.
idebr → created an issue.
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
@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
The test checks for the aria-describedby attribute. This attribute is only available when the field has a description.
The remaining deprecations are now fixed in the merge request
The merge request updates configuration forms to use #config_target
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 →
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
Considering Drupal 11 has been released a while ago, this issue can now be marked as fixed?
The merge request was not correctly linked to this issue. It is available at https://git.drupalcode.org/project/consumers/-/merge_requests/20
This issue also happens to be caught by phpcs, so I fixed these issues in a single go
The merge request removes lingering 'priority' keys from node access records
🌱 Move to thephpleague/oauth2-server 9.0 Active has been committed. However, there is no upstream support yet for introspection
The change is available in 📌 Fix phpunit test Active for 6.0.x
More test failures due to 📌 Define 'original' as property on the entity object Needs work
Test failure due to 🐛 Password and confirm password should be mandatory fields while setting up password using one time link following by email Needs work
The merge request adds the following changes:
- The configuration form now uses #config_target
- The default configuration syntax now matches the config when first pressing 'Submit configuration'
In the merge request ctools is no longer installed when testing pathauto
I converted the patch to a merge request: https://git.drupalcode.org/project/tmgmt/-/merge_requests/86
The change works as expected.
idebr → made their first commit to this issue’s fork.
Sure, that makes sense. Can completing the test coverage be done in a follow-up issue in 2.3.x?
@steffenr The merge request seems pretty complete. Is there anything that needs to be changed before a merge? The official PHP client would be very welcome when working on ✨ Support translation of documents Active
The conversion exposes a bug in the config schema where tfa.settings:mail is defined as a config object:
Symfony\Component\Validator\Exception\LogicException: The LangcodeRequiredIfTranslatableValues constraint is applied to 'tfa.settings::mail'.
This should be fixed in this issue to make the tests pass
greggles → credited idebr → .
gitlab-ci.yml was added in 🐛 Kernel Tests: Errors and Deprecations Fixed