🇧🇪Belgium @Mschudders

updated to latest DEV.

Mschudders → created an issue.

Mschudders → created an issue.

Mschudders → created an issue.

example on how to post media:

$this->twitterPostManager->setClient($client)->doPost([
'status' => 'Le Label',
'media_paths' => [
$mediaPath, >> Should be a FILE URI.
],
]);

If this gets applied (this merge request) >>> please update documentation.

This incoperates the patch from: https://www.drupal.org/project/social_post_twitter/issues/3392216 🐛 Authentication works but profile isnt returned 404, Warning: Attempt to read property "id" on null in RTBC
So be sure to give credits to https://www.drupal.org/u/zarexogre →

That issue was about making it work again, this is about yeah applying media/images.

Mschudders → created an issue.

merge request provided

Mschudders → created an issue.

testing okay after final bug fix

put to needs review

Mschudders → created an issue.

I tested this and it works.

Moving to needs review

patch added and merged manually.

updated project page: https://www.drupal.org/project/views_filter_select →

Marking this as fixed.

postponed

Well you need to fill in javascript selectors on the settings page. I think by default there are "defaults/placeholders" filled in at the settings page?
The selectors specify a certain element on for example the search page.

Sorting for example: You need to write a javascript selector that targets this element, for example a "select" element.

Does this help you?

fixed

Can you elaborate? The module is still available and supported so I don't know what is wrong atm?

Thanks for clarifications.

closing, merged to DEV.

merged thanks!

resolved merge conflicts, all good. fixed, thanks!

Mschudders → made their first commit to this issue’s fork.

so the fix has been reverted. (Checking commits on Merge request)

Is the orderBy still needed then or can we close the ticket? Thanks.

closing

merged. Thx

Mschudders → created an issue.

Mschudders → created an issue.

merge request= https://git.drupalcode.org/project/entity_share/-/merge_requests/89

put to needs work >>> For testing ;-)

code pushed, will test.

Mschudders → created an issue.

added to DEV.

Mschudders → created an issue.

added in DEV. Copied file from admin_toolbar_version.

committed to latest DEV.

Thanks!

Mschudders → created an issue.

committed to branch. > Needs review

Mschudders → created an issue.

Mschudders → created an issue.

Mschudders → created an issue.

Hello,

Is the project overview page clear enough for you: https://www.drupal.org/project/views_filter_select → ?

Otherwise I will elaborate.

Thanks

done.

Closing

this is not necessary anymore as of 10.2.x

There is already a fix inside of the JS. Nicely done.

fix pushed.

Ongoing testing.

todo have to test mixing "+" and ",".

Right now tested "+" and "single values"

Before:

 if ($value) {
            $storage = \Drupal::entityTypeManager()->getStorage($entity_type);
            $entity = $storage->load($value);
            $form['override'][$block->getDerivativeId()][$type][$id]['value']['#default_value'] = $entity;
          }

After:

 if ($value) {
            $storage = \Drupal::entityTypeManager()->getStorage($entity_type);

            $values = preg_split("/[\+,]+/", $value) ?: [];
            $default = $storage->loadMultiple($values);
            $form['override'][$block->getDerivativeId()][$type][$id]['value']['#default_value'] = $default;
          }

Mschudders → created an issue.

Mschudders → created an issue.

Mschudders → created an issue.

I can also confirm #39 works on a big Drupal multilingual Drupal 10.1.5 with Apache (no NGINX)

update issue to 10.1.x

Mschudders → made their first commit to this issue’s fork.

all good

I would refactor the naming of the methods instead of declaring this as a conflict.

Doesn't hurt even though.

Will do a refactor later this year so this will get picked up in there (normally)

pushed onto DEV.

TODO test

Mschudders → created an issue.

Adding screenshot which will help in debugging/finding the issue:

Mschudders → created an issue.

Mschudders → made their first commit to this issue’s fork.

Mschudders → created an issue. See original summary → .

Mschudders → created an issue. See original summary → .

fixed

Mschudders → created an issue. See original summary → .

Please review new patch for latest 2.0 version.

Mschudders → made their first commit to this issue’s fork.

fixed

fixed

Mschudders → created an issue.

Mschudders → created an issue.

fixed on latest DEV.

Mschudders → created an issue.

which version of layoutbuilder_extras are you using?
This was resolved in 2.X (I think will check)

+1 to opt into security coverage.
You can do this on the project page if the project is "old" enough

closing ticket.

Merged. Thanks!

fixed in dev.

Going to add to 1.0.7

Mschudders → created an issue.

Mschudders → created an issue.

Can you import the config manually? (views.view.....yml)

Or do an uninstall - install?

I've added this to the latest DEV branch.

D10 version not yet tested ;-)

Mschudders → made their first commit to this issue’s fork.

Mschudders → created an issue.

putting status to closed

Merged thanks!

resolved on latest DEV mate.