๐Ÿ‡ฎ๐Ÿ‡ณIndia @Jeya sundhar

Coimbatore
Account created on 15 August 2018, over 5 years ago
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

This Issue fixed in 3.0.x-dev

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Jeya sundhar โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

I checked #19 patch. that patch not worked for me. few changes based on Stackoverflow answer.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

I am nominating myself to be the maintainer for this module, primarily because I require a Drupal 10 version for a client site. Additionally, I have a solid understanding of this module.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

composer.json changes

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

:+1 Reviewed for flags and dropdown thats working fine.
Thanks @naveenvalecha

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Jeya sundhar โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Jeya sundhar โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi all,
I reviewed this branch(3297166-D10-compatible).
CSS & JS Directory not created

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @xpersonas
You need to reorder in manage form display

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Oh! mistakenly tested 8.x-3.x
This branch doesn't show that issue

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

after using this issue repo
D10 site throws
```

TypeError: Drupal\htmlmail\Plugin\Mail\HtmlMailSystem::__construct(): Argument #10 ($mimeTypeGuesser) must be of type Symfony\Component\HttpFoundation\File\MimeType\MimeTypeGuesserInterface, Drupal\Core\ProxyClass\File\MimeType\MimeTypeGuesser given, called in /modules/custom/htmlmail/src/Plugin/Mail/HtmlMailSystem.php on line 147 in Drupal\htmlmail\Plugin\Mail\HtmlMailSystem->__construct() (line 109 of modules/custom/htmlmail/src/Plugin/Mail/HtmlMailSystem.php).
```

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @andre.bonon
Thanks for the patch. :+1

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Jeya sundhar โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @Pol Dellaiera :)
$uri used instead of $url. Coding standard fixes updated :+1

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Jeya sundhar โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @Pol Dellaiera
Proposal of something smaller without almost no local variables changes updated. :+1

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @Pol Dellaiera
Thanks for the review.
Condition updated. file_exists checked before getting image size. :)

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

In composer.json add following repository

        {
          "type": "package",
          "package": {
            "name": "drupal/openapi_jsonapi",
            "type": "drupal-module",
            "version": "dev-3.x",
            "source": {
              "type": "git",
              "url": "https://git.drupalcode.org/issue/openapi_jsonapi-3353029.git",
              "reference": "743172b90a26208653e39835749d3524219c32b3"
            }
          }
        },

before

        {
            "type": "composer",
            "url": "https://packages.drupal.org/8"
        }

Then run composer require drupal/openapi_jsonapi:dev-3.x

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Jeya sundhar โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @longwave Thankyou. After composer update. I reboot the aws server. It's now updated to 9.5.8 ~> 9.5.10

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

This changes available in next release 1.0.5

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

+1 Reviewed code commited to dev branch

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

I would like to help to co-maintain this theme and I recently worked in the subtheme of Adminimal theme. and I need to port this theme to Drupal 10 ASAP.

Thanks,
Jeya Sundharam

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @sandip.prajapati,
Will you share your exported view (D7) for reference to fix this issue.

Thank you,
Jeya Sundharam

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

It's not Related views migration.
"Couldn't upload file:" - This issue related in folder permission. Create Files folder with 777 permission

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @skounis
This Issue fixed in parent issue. once test with dev site.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Jeya sundhar โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Re-roll Patch
Apply this patch after applying patch metadata-support-for-payment-intent-3215882-6.patch โœจ Metadata support for payment intent Needs review

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @Qaiser iqbal,
Will you share your exported view (D7) for reference to fix this issue.

Thank you,
Jeya Sundharam

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Hi @sidharth_soman ,
This merge only contains src/Plugin/PluginEventSubscriber.php , Need to update views_migration.services.yml views_migration.plugin_event_subscriber service arguments.

Thank you :)
Jeya Sundharam

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Jeya sundhar โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

:+1 RTBC
@thomas.lobjoie
Thanks for this patch

๐Ÿ‡ฎ๐Ÿ‡ณIndia Jeya sundhar Coimbatore

Facing Issue on Drupal 9 to Drupal 10 core update

Production build https://api.contrib.social 0.62.1