Account created on 22 May 2011, about 14 years ago
#

Merge Requests

More

Recent comments

Got the same error on production on Drupal 10.4.7, seems related:
https://www.drupal.org/project/gin/issues/3509501 🐛 User warning: Trying to overwrite cache redirect Active

Darren, I get a warning once I grant a user any permission on a node:

Warning: Undefined array key "grant_view" in Drupal\nodeaccess\Form\GrantsForm->validateGrants() (line 224 of modules/contrib/nodeaccess/src/Form/GrantsForm.php).

Test wise:
Warning: Undefined array key "grant_view" in Drupal\nodeaccess\Form\GrantsForm->validateGrants() (line 224 of modules/contrib/nodeaccess/src/Form/GrantsForm.php).

That aside, there are room for code improvements as well

It looks good tbh and I reviewed these two, which is great work from Darren:
- https://www.drupal.org/project/nodeaccess/issues/3357718 🐛 Warning: Undefined array key "rid" + Trying to access array offset on value of type null in nodeaccess_node_access() Active
- https://www.drupal.org/project/nodeaccess/issues/3144313 Preserve role grants when adding custom grants on node Needs work

IMO, we can merge the two and tag a new alpha with D11 support.

I made a proof of concept of what I mean, we can make this configurable so it would load the Gin based design or the previous approach.

That aside I believe it would be nice if we have another configuration to have this as a toolbar item rather than a block in a region.

Thanks Jukka, I need to further understand the ecosystem myself. My use case is that I already have a "CKEditor automated translation job" that uses OpenAI to translate text based on another field in the node. it works fine and I wanted to switch from OpenAI to another provider. But from what I understood, I can't really do that since the CKEditor text seems to be a "Chat with Complex JSON" scenario.

Thanks a lot for a quick reply.
It just says provider_deepl.settings already exists.

That aside, a side question, do we have AI automators integration? I want to use "Chat with Complex JSON" with DeepL provider.

Thanks for the PR, but by enable, you mean setting the localStorage? Not sure if I see the benefit. For instance you prefer if it stays open on all pages whenever you are moving around the website?

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

Great, thanks a lot for the MR, merged. Hopefully will tag the last release

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

Thanks but there's a merge conflict as of now it seems

Glad to hear that, I was starting to doubt that decision, good to get it confirmed

I'm not sure if you are filing the issue under the correct branch or not Thomas but the 6.x does not have a proxy.js file but rather .env.local file, you might have mixed things up with 5.x documentation

Hey Mikey, thanks a lot for the MR and the Gutenberg work, merged.

This causes the error mentioned above and I kinda don't see any way around recovering this

Nothing on my end, not actively working on nodeaccess at all, but I'd say let's give it a bit and see how things are and if all is good we can tag a new release

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

Thanks Rajab for the update, I'll mark this as closed for now since it's not Radix related.

I have no idea Pierre, I haven't looked further into it but you can test on D11, generate some content with devel on let's say olivero, check the content generated and some throw the error. Works after re-saving it though

@Stephan and @Pierre, I looked a bit further into this, and it turns out at least based on the error I am getting this is not a Radix nor UI patterns issue but rather the D11 core I suspect.

So the error exists on Radix, Olivero, and any other SDC themes, it gets fixed if you just re-save the node.

Stephan, could you check please, and see if that's the case?

@Stephan I managed to recreate it with Drupal 11:

TypeError: Drupal\Component\Utility\Html::escape(): Argument #1 ($text) must be of type string, null given, called in /var/www/html/web/core/lib/Drupal/Core/Utility/LinkGenerator.php on line 198 in Drupal\Component\Utility\Html::escape() (line 431 of core/lib/Drupal/Component/Utility/Html.php).

I'll keep you guys posted on a fix.

@Stephan I somehow couldn't recreate the errors you mentioned on Drupal CMS. I'll try with Drupal 11 Core to see if that's the case. I suspect it might be a content-specific error.

Would be nice to know what error you were getting.

@Pierre still on my todo list to clean up the remaining components, definitely will give the SDC devel a go, thanks a lot for the help.

It's been a while since I last looked into this and I don't recall the reason for it Tom, I wonder if it's a core issue but we'll investigate, thanks

Thanks a lot, appreciate the work. I am not knowledgeable here as well, so I guess this is as good as it gets for now.

Thank you, I'll make sure to refer to our conversation when I'm on a call with my therapist. Cheers

Btw, instead of being hostile and calling your co-maintainer someone who stole your code, you could've just said, let's revamp 1.x instead of starting new fixes on 2.x 🙂

Something to think about for the future.

Begging is a big word, especially when I emailed you and told you to work together, wow, you have some issues, instead of reading on git branches please consider a therapist.
To wrap this up:
- So your only issue is that I opened a new branch instead of continuing on the 1.x branch? and you consider that as stealing? just a new branch?

Anyway, don't waste more of my time, best of luck :)

Marking this as Closed (due to lack of thinking capabilities)

Daniel, please consider what you are saying:
- I stole your work with a fix and introducing a new branch?
- You (Yes you have access as a maintainer to all the branches) could've also just merged the 2.0.x to 1.0.x if you wanted to, even though that's wrong, you should just start a new branch like I did :)
- Also did you reported me for creating a new branch :)))

Dear Daniel,
I'm really struggling with your lack of knowledge on how things work, a couple of things:
1. Your code is already open-source
2. I joined your project as a co-maintainer, created a new branch called 2.0.x and fixed this issue https://www.drupal.org/project/gutenberg_ai_tools/issues/3473966 Please consider using the Ai module Active
3. You sent me an email saying that I stole your code? I'm your co-maintainer FFS :)) If I am to steal your code, would I add a new branch on the same project? You know we both have access to all the branches on a project right?
4. The reason for a new branch is, your code is dependent on obsolete OpenAI module, I integrated it with the AI module, so backward compatibility requires you to create a new branch (please study on that)
5. AGAIN IT WAS STILL ON YOUR OWN PROJECT WHICH YOU HAD ACCESS
6. You kept asking me to opt-in your "Alpha" release into security coverage, you need to be stable first to even try to opt-in
7. Please study on git branches and reasons people branch out and backward compatibility

Marking this as "needs review"

Thanks Stephan, got rid of the whole thing

This is causing too many issues, I just pushed a fix by removing the type limitation on the rows. should be good for now but not the ideal solution.

Please test and let me know if it helped

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

Great idea, could you please provide a MR maybe? if not let me know

Hey Dan, you are correct, I missed the breaking change part, it definitely is. We should ideally keep it and also add the new d-flex class.

Thanks for the updated MR James

Any update on this? it's been on production for me for a while and works as expected

Thank you Jon for this, could you please provide the MR?

Thanks James, I'm not sure if flex-row and flex-wrap are needed, are they? can you test without? if so update the MR?

Hey James, Thanks for a nice find, please provide the MR, I appreciate it, thanks

Hey guys 👋

I have a long todo list to add to this module but unfortunately, don't have a lot of free time, I'll see if I can open up sometime if possible.

This is correct as is, changing the background color of Gutenberg parts is out of admin theme control. If you want to change the background color of Gutenberg, you need to take care of that outside of the admin theme.

Marking this as Closed for now, feel free to re-open

I'm unsure if this is the correct fix, even though it doesn't hurt to apply your patch and help other people with an extra headache, I assume Mercury editor should pass an empty '' string instead of null. Maybe add a ticket over at their issue queue as well.

Regardless I applied your patch since it's okay to have this, thanks.

Thanks again Phil, merged. I also updated the docs.

Ideally, we should just check and tag a D11-compatible version of 4.x and 5.x
Not necessarily supporting it but rather making it possible to upgrade.

Hey Dan, this sounds about right to me, at some point, I slowly tried to convert them but seems like I never finished. That aside, I don't believe there would be any breaking change with this, so feel free to get this going.

Thanks a lot

Yes, I just haven't gotten to it yet, sorry. Could you maybe provide a MR? if not I'll check soon

No. But you can always Sub-theme any theme yourself.

Okay I pushed a backward compatibility fix for this since one of my projects also got hit with this. marking this as fixed for now, and taggign a stable release 6.0.0 🎉

Perfect Dan, I wanted to merge it but seems like MR is in draft

Hey Dan,

A couple of notes on my end regarding the issue:

> Currently when a radix template changes, existing sites will not receive the change without manually copying the change from the starterkit to their sub-theme.

That's by design, and I believe that's okay to be like this, any older website that is already built, shouldn't be needing to change their markup (eg. templates and so on), so what they currently have should suffice. If they are keen on being updated, they should manually review and update if needed.

> When the change is accompanied by a corresponding change to the component, this can cause incompatibility where none would otherwise occur.

This is the tricky part, but we've been doing this since 8.x.4.x more or less, we just need to be more mindful of backward compatibility when changing the components.

> Also, core discovers the templates in the starterkit and uses them in the absence of a template in the sub-theme.

That's the core problem to be addressed, when we set the starterkit: true it should just ignore the templates, but for now let me take a look, I might have some solutions for this, eg. fully remove the starterkit once it gets copied

Even though this is okay to add the issues here, if anything we should add the related issues to Github repo: https://github.com/doxigo/drupal-radix-cli

That aside, glad your issue is fixed once you updated your node path, I can look into this further see if it's something we can help with or not.

Okay we will definitely will have unhappy users, but what's done is done. I suppose we can keep the 6.0.x since in Drupal, 6.1 and 6.0 is not that different. Also we are still on RC, so we have an excuse :)

to add my two cents:
If we make a change that has breaks things for existing websites, we should do a revert for now. if it's a major change that we can't live without, we can do a new major release even and move forward with that. Otherwise not all people will find this thread.

Production build 0.71.5 2024