Santa Cruz, CA, USA
Account created on 16 April 2004, about 21 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

This is still an issue... also can't click on the card when it's in there to get its properties.

You have to use Layers to get cards in place and edit properties.

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

kristen pol created an issue.

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

While CKEditor is working, it is hijacking the styles, so that the theme (light/dark) and style props aren't working... so something else is needed

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

After I went to the enabled component list, they showed up so closing this

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Does that branch help this issue get back into scope?

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

I read the subject in the queue and knew exactly who must have filed this issue :D

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

I updated the docs and everything seemed to install fine but the SDDS components are not showing up in XB so I've pinged the Slack channel to see if anyone has any thoughts

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Weird... when I deleted the code component, it shows the placeholder text... hmm

Previously, I had MOVED the code component into the regular components area, and I swear it didn't show anything

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

remove XB patch step

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

I’m not opposed to the logo… it just doesn’t feel like “Experience Builder” to me… but not sure what would. I’m NOT a designer:)

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

The starshot-paragraph component is another good one where this could be added

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Oh... haha:

https://twig.symfony.com/doc/3.x/filters/striptags.html

You can also provide tags which should not be stripped:

{{ some_html|striptags('<br><p>') }}

That is not obvious DX :D

Okay... then this seems fine. I'll merge

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

I see this is the correct information in the yml as noted here:

https://www.drupal.org/project/experience_builder/issues/3519135#comment... Use CKEditor 5 for text area prop's example form element in code editor, adjust prop shape sent to backend Active

I don't understand the striptags though... you are stripping out all the links and bold and such? Doesn't this defeat the purpose of the CKEditor area? Or maybe I haven't had enough caffeine yet

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Thanks 🙏 It’s very clean but I’m not sure it’s clear what it represents. I’ll wait for others to chime in

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

This has been merged for both 1.x and 2.x

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Tested on D10 on 1.x branch and one thing I noticed (which is probably still an issue for D11/2.x too)... if you have it set to push unpublished content, you don't get it because that setting isn't being honored in this code. But that's a separate bug, so we can handle it separately

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

I tested on D11 with translated unpublished taxonomy term without the patch and the term was pushed to quant.

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

On D11, when testing without the changes, the unpublished translation doesn't get pushed over, but this error shows up in the logs:

Path: /af/node/12. Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException: in Drupal\Core\Routing\AccessAwareRouter->checkAccess() (line 115 of /var/www/html/web/core/lib/Drupal/Core/Routing/AccessAwareRouter.php).
🇺🇸United States Kristen Pol Santa Cruz, CA, USA

That’s from the demo design system

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

One thing that needs updating is probably changing Vienna2025 to just an “AI Initiative” tag so it’s easier to collect issues across projects

Will update by tomorrow

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Removing "community" based on call today... the focus isn't necessarily just internal within the community, but also potential users who are outside the community

🇺🇸United States Kristen Pol Santa Cruz, CA, USA
🇺🇸United States Kristen Pol Santa Cruz, CA, USA

The PR needs to be updated based on my comments in GitHub

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

The code is updated so that Seed::seedNode() uses the translated entity in two areas, so that the correct entity data is available, e.g., published status:

  • quant_cron_cron
  • NodeItem::send

To test the first, we need to use cron like:

  1. Do not sync upon content edit
  2. Translate a node and make it published
  3. Look in Quant and the content will be missing
  4. Run cron and it will show up in the dashboard
  5. Repeat with the translation as unpublished and you won't see the content in dashboard after running cron
  6. Change the quant settings to sync unpublished content and run cron again and then it will show up in the dashboard
🇺🇸United States Kristen Pol Santa Cruz, CA, USA

kristen pol created an issue.

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Thanks. @phenaproxima. This is very helpful!

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

kristen pol created an issue.

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Merged and checked it's not used elsewhere and it's okay

Before

./src/Settings/CivicthemeSettingsFormSectionBase.php:use Drupal\Core\Messenger\Messenger;
./src/Settings/CivicthemeSettingsFormSectionBase.php:   * @param \Drupal\Core\Messenger\Messenger $messenger
./src/CivicthemeConfigImporter.php:use Drupal\Core\Messenger\MessengerInterface;
./src/CivicthemeConfigImporter.php:   * @param \Drupal\Core\Messenger\MessengerInterface $messenger

After

./src/Settings/CivicthemeSettingsFormSectionBase.php:use Drupal\Core\Messenger\MessengerInterface;
./src/Settings/CivicthemeSettingsFormSectionBase.php:   * @param \Drupal\Core\Messenger\MessengerInterface $messenger
./src/CivicthemeConfigImporter.php:use Drupal\Core\Messenger\MessengerInterface;
./src/CivicthemeConfigImporter.php:   * @param \Drupal\Core\Messenger\MessengerInterface $messenger
🇺🇸United States Kristen Pol Santa Cruz, CA, USA

I misunderstood this issue and thought it was for the initiative as a whole rather than just marketing. I had some minor feedback but put it in slack but I’ll try to edit this week when I’m back at my computer

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Thanks!

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Postponing for now due to more pressing priorities

🇺🇸United States Kristen Pol Santa Cruz, CA, USA
🇺🇸United States Kristen Pol Santa Cruz, CA, USA

closing

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Add credits

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Moving to initiative

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Moving this to the initiative

📌 | AI | Landing Page
🇺🇸United States Kristen Pol Santa Cruz, CA, USA
📌 | AI | Landing Page
🇺🇸United States Kristen Pol Santa Cruz, CA, USA

This was added as part of the AI initiative launch in June at drupal.org/ai (along with subpages)

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Now that drupal.org is being used for pages for AI content, we don't necessarily need this

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Documentation issue:

📌 Document new AI issue management process Active

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

kristen pol created an issue.

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Postponing for now due to more pressing priorities

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Postponing for now due to more pressing priorities

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Ones that I had proposed before in various docs are webinars on:

  • Roadmap
  • Funding
  • Contribution
  • Marketing
  • Regular status updates (monthly)
🇺🇸United States Kristen Pol Santa Cruz, CA, USA

This is better for Paul as he's already running with this

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Updated the top of the project to point to /ai along with this project, the AI core project, and the AI ecosystem list.

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

@pdjohnson please provide any thoughts on this and then assign to Matthew for thoughts and then it can go back to me

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

I've done the first round of issue cleanup... there can be more done, but it isn't a priority:

https://www.drupal.org/project/issues/search/ai?project_issue_followers=...

I've discussed where the new docs should live with Marcus, and we've agreed they should be part of the existing docs in GitLab:

https://project.pages.drupalcode.org/ai/1.1.x/contribute/

Note that to add images, you have to add them to the wiki first:

https://git.drupalcode.org/project/ai/-/wikis/home

which requires special access

An example docs page with an image:

https://project.pages.drupalcode.org/ai/1.1.x/agents/orchestration/

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

unassigning

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

unassigning

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Change component

🇺🇸United States Kristen Pol Santa Cruz, CA, USA

Change component

Production build 0.71.5 2024