- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
Merged into 1.x and cherry-picked to 1.1.x, thanks!
-
penyaskito →
committed 33cc7f85 on 1.1.x authored by
darren oh →
Issue #3515847 by darren oh, marcus_johansson: Add environment variables...
-
penyaskito →
committed 33cc7f85 on 1.1.x authored by
darren oh →
-
penyaskito →
committed a23f4aba on 1.x authored by
darren oh →
Issue #3515847 by darren oh, marcus_johansson: Add environment variables...
-
penyaskito →
committed a23f4aba on 1.x authored by
darren oh →
- First commit to issue fork.
- 🇩🇪Germany marcus_johansson
Looks good to me, we added it to the config action as part of some hosters wanting to setup the AI Recipe using env vars, so its good its being reused here.
I've tested so there is no regeression, either on GUI or terminal side and so that you can install it with a set environment key in ddev.
- 🇬🇧United Kingdom MrDaleSmith
The MR is in draft: it would be good to remove that if you have finished with it, regardless of the maintainers decision about whether it can be merged yet.
Applies cleanly, works well for meL: I *much* prefer this flow which makes a lot more sense than the current one.
- 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Thanks 🙏 We will be making a website as well so we should figure out what’s where
- 🇩🇪Germany breidert
First version was created and all plugins were updated.
It looks like this:
However, this might be a breaking change, because all existing plugins would have to be updated.
- @breidert opened merge request.
- Issue created by @breidert
- 🇳🇿New Zealand quietone
The changes read well.
This could still use a screenshot.
- 🇫🇷France goz
Looks great.
What's about a test to confirm props attributes will keep its values in case keys exist in both.$build = [ '#type' => 'component', '#component' => 'sdc_theme_test:my-card', '#props' => [ 'header' => 'Drupal.org', 'attributes' => new Attribute([ 'foo' => 'bar', ]), ], '#attributes' => [ 'foo => 'third', ], ];
I think in this case, foo should still be bar.
And what's about attribute with array like classes for example ?
$build = [ '#type' => 'component', '#component' => 'sdc_theme_test:my-card', '#props' => [ 'header' => 'Drupal.org', 'attributes' => new Attribute([ 'foo' => ['bar', 'ter'], ]), ], '#attributes' => [ 'foo' => ['quater'], ], ];
-
balintbrews →
committed bd499944 on 0.x
Issue #3515903 by balintbrews, lauriii, gábor hojtsy: Add starter code...
-
balintbrews →
committed bd499944 on 0.x
- 🇫🇮Finland lauriii Finland
Looks good! It would be nice to make this customizable in future but this is a great improvement so I don't think we need to hold this back on that.
- @balintbrews opened merge request.
- 🇳🇱Netherlands balintbrews Amsterdam, NL
Vision: Lauri. Serious wordsmithing: Gábor.
- Issue created by @balintbrews
I could not recreate this issue in Drupal 11.x-dev, and it appears fixed using CKEditor 5. As the screenshot shows, in a custom content type I added a content block using layout builder and was able to upload an image to the text field. @mmenavas helped me with this issue at Drupalcon.
- 🇺🇸United States jdleonard Austin, TX, USA
Updated issue summary based on consensus at DrupalCon Atlanta 2025.
- Issue created by @darren oh
- 🇺🇸United States ivagold Philadelphia, PA
I've changed the case in that page's title to match the link in the menu. Is that the only thing that needed updating?
- 🇺🇸United States phenaproxima Massachusetts
tim.plunkett → credited phenaproxima → .
- 🇺🇸United States tim.plunkett Philadelphia
Discussed this with @leslieg, @jquijano, @mshaik, and @phenaproxima.
Adam and I suggested that we move AU to require from require-dev, so that it is always in the codebase. Then the message can link to the module install page. But it retains the ability for people to NOT use Package Manager.
- 🇺🇸United States tim.plunkett Philadelphia
tim.plunkett → made their first commit to this issue’s fork.
- 🇪🇸Spain plopesc Valladolid
Good progress so far! :)
Added a minor comment to the MR to reduce a bit the file footprint. Besides that, would be great if the Navigation settings form page could be updated as well. It is defined in
navigation.routing.yml
file.Thank you!
- @darren-oh opened merge request.
- 🇺🇸United States ivagold Philadelphia, PA
Ok, It's updated, and when I test it, the navigation menu items are reordered and match in text case:
- Issue created by @darren oh
- 🇺🇸United States chrisfromredfin Portland, Maine
Just confirming that Package Manager is not a strict _requirement_ for Project Browser; PB functions without it (though not in as wonderful a way). But it is a soft dependency.
- @mshaik opened merge request.
- @ivavictoria opened merge request.
- 🇺🇸United States ivagold Philadelphia, PA
I'm at DrupalCon Atlanta and am interested in giving it a try! Let's see how far I get. ;-)
- 🇲🇽Mexico dalin 🇨🇦, 🇲🇽, 🌍
Since Package Manager is a requirement for Project Browser, we should list it as a hard dependency (in the .info.yml file) now that it is included in Drupal Core.
- Issue created by @RobLoach
- 🇺🇸United States trackleft2 Tucson, AZ 🇺🇸
I can confirm this is an issue, and found that the minimum Drupal core version that contains packageManager by default is 10.2.5
See https://git.drupalcode.org/project/drupal/-/blob/10.2.4/core/package.jso...
vs
https://git.drupalcode.org/project/drupal/-/blob/10.2.5/core/package.jso...I think we can remove all of the help text that references packageManager
- Issue created by @danieazc
- Issue created by @danieazc
- Issue created by @danieazc
- Issue created by @danieazc
- Issue created by @danieazc
- Issue created by @danieazc
- Issue created by @danieazc
- Issue created by @danieazc
- @mshaik opened merge request.
- 🇺🇸United States mradcliffe USA
Potentially a redirect from "documentation/is-drupal-secure" the Frequently Asked Questions page → .
- @dipakmdhrm opened merge request.
- 🇺🇸United States alyssavoccia
I found that this page looks to be duplicated
https://www.drupal.org/documentation/is-drupal-secure → -> https://www.drupal.org/features/security#:~:text=Many%20security%20probl... → . - Issue created by @jdleonard
- 🇺🇸United States sundarcreosen
I am working on this issue during DrupalCon Atlanta 2025.
- 🇺🇸United States ijf8090
Auto updates was released as a contrib module with Drupal 10 and became part or core with Drupal 11.1 release.
- 🇺🇸United States alexdmccabe Orlando, FL, US / Seminole lands
I created a document in the Google Drive folder. I'll add the link to the issue description, but it's also here: https://docs.google.com/document/d/1UJ9S6V6mS_82UG64VE6EcQAMoEMO90Cys7gB...
I'm working on this issue at DrupalCon Atlanta 2025. I'll be working on this today with TrackLeft2
working on this issue as my first contrib at DrupalCon 2025 Atlanta. Being mentored by trackleft2
- First commit to issue fork.
I am going to work on this one as part of first time contrib in Atlanta.
- Issue created by @chrisfromredfin
- Issue created by @jdleonard
- Issue created by @jdleonard
- Issue created by @jdleonard
- Issue created by @thejimbirch
- Issue created by @rgladden
- Issue created by @jdleonard
- Issue created by @ChrisDarke
- Issue created by @mradcliffe
- Issue created by @pdureau