🇺🇦Ukraine @nginex

nginex → created an issue.

nginex → created an issue.

going to implement a hook_update_N to update this setting if file_enity module is enabled

also I can add a warning message on content import page

Please add application/zip to mime types of document at /admin/structure/file-types/manage/document/edit

This should solve the issue

nginex → created an issue.

nginex → created an issue.

I've provided support of uuids along with ids, so both values are supported at the same time, see MR

nginex → created an issue.

I'm currently working on a solution where creating files will not be needed.

It will use StreamedResponse instead of downloading file.

For now I think I can provide a patch where the file schema can be configured. e.g. temporary, public, private and etc

Could you please ensure that temporary folder is properly configured on stage and live? Do you use external file storage?

It works like a charm

I also dropped Drupal 8 support

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

Guys,

Please do not refactor/fix coding standards of non-related code to this issue, you are causing at least a merge conflict and a lot of mess, this is not a scope of the issue, it's a purpose of a separate issue in the project

Instead of using a link tag to attach css file, it's better to expose the css into style tag, it should work like a charm.

/**
 * Implements hook_preprocess_HOOK() for entity_print.
 */
function mytheme_preprocess_entity_print(array &$variables) {
  /** @var \Drupal\Core\Extension\ModuleExtensionList $module_list */
  $module_list = \Drupal::service('extension.list.module');

  /** @var \Drupal\Core\Extension\ThemeExtensionList $theme_list */
  $theme_list = \Drupal::service('extension.list.theme');
  $css = [];

  // Expose default css into style tag.
  if (\Drupal::config('entity_print.settings')->get('default_css')) {
    $css[] = @file_get_contents($module_list->getPath('entity_print') . '/css/entity-print.css');
  }

  // Expose custom css into style tag. In my case it's located in dist/pdf-styling.css.
  $css[] = @file_get_contents($theme_list->getPath('mytheme') . '/dist/pdf-styling.css');

  $variables['entity_print_css'] = [
    '#type' => 'inline_template',
    '#template' => '<style>{{ output }}</style>',
    '#context' => [
      'output' => implode(PHP_EOL, array_filter($css)),
    ],
  ];
}

first test run used php 7.3, it's not supported, so I rerun with another php version, all good

works like a charm

It's now available in 2.2.3,

Thanks all

I will take a look

Hi Joe,

It was already fixed in this issue #3293630: Consider dropping \d+ requirement in the export route → , and the fix is available since 1.3.6, please upgrade the module or apply the patch.

As alternative option (without updating to the new version), you can add this to your composer.json

"replace": {
    "drupal/action": "*"
}

nginex → created an issue.

Available in 6.1.6

Available in 6.1.6

@trafo, the MR contains everything but the changes that needed, why the are 36 commits? Do you know what is a rebase and how to work with it?

Please fix notes in the MR before sending to Needs review again

The button is hidden by default, so there should not be breaking changes. At least there is now a possibility to style the button as a play button and it's accessibility friendly

The patch has issue with tabs/spaces

@VincentL, please stop adding patches that does not belong to module code

Available in 2.1.2

I think this should be improved in doImport() method, at least we can have an extra validation by loading users by email and username if entity by uuid was not found. I will provide a patch, once it's available, there will be nothing to do on your side, I'm also planning to provide a new release soon, so the patch will be included to the new release immediately

Provided a better explanation how to import a zip file

This is now available in 2.0.1

This was done since 2.0.0 release

Thanks guys for patching and testing

I wish I could create a MR against 2.0.x but dev release was not created for that branch, so it's not possible to do so, I will attach a file patch

nginex → created an issue.

You need to use $entity->getDescription() as a value and $entity->getFormat() as a format, currently the format is hardcoded on your patch

nginex → changed the visibility of the branch 3030711-views-ajax-handling to hidden.

The MR looks good, it handles one more bug

+1

bohart → credited nginex → .

Fixed typo, added more details

Great idea,

I've added it here

https://www.drupal.org/docs/extending-drupal/contributed-modules/contrib... →

Fixed typo

Added Troubleshooting

Hey @Dency Mary A

Can you check again? I've provided a fix in the 3.0.0-rc5 release

composer require drupal/cloudinary shoud mormally work

Hello,

As workaround, try to import a config manually:

1. Copy content of https://git.drupalcode.org/project/single_content_sync/-/raw/1.4.x/confi...
2. Go to /admin/config/development/configuration/single/import
3. Choose Action configuration type
4. Paste the content of the config and click on import

Thanks for the patch, I will include it to a new release in a moment

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

nginex → created an issue.

nginex → created an issue.

Reroll for 2.1.x

integrate with single_content_sync

integrate with single_content_sync

Thanks for the patch, I found a few issues with options description, you can see it when running drush content:export --help

Need to fix it, and then the issue can be resolved

nginex → changed the visibility of the branch 3325903-if-author-doesnt to hidden.

nginex → changed the visibility of the branch 1.4.x to hidden.

Well done!

nginex → created an issue.