Add core_version_requirement: ^8 || ^9 professional_theme.info.yml to designate that the module is compatible with Drupal 9. See https://www.drupal.org/node/3070687.

Comments

kishor_kolekar created an issue. See original summary.

kishor_kolekar’s picture

Status: Active » Needs review
StatusFileSize
new365 bytes

please review the patch.

kristen pol’s picture

Issue tags: -D9 porting day +Drupal 9 porting day
kristen pol’s picture

Actually, porting day was April 28th so fixing tag.

indrajithkb’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new36.2 KB
new28.2 KB
new391 bytes
new331 bytes

Hi @kishor_kolekar thanks for the patch, it fixing the issue.
But i have re-rolled the patch(was little bit confused that's why i updated) .but we need both(corecore: 8.x and core_version_requirement: ^8 || ^9) otherwise theme will not work with the versions which is less than 8.7.7.
so we can go with the #2 patch.

Attaching screenshots for reference.

Before patch:

before patch

After fix

after patch

indrajithkb’s picture

am just hiding my #5 patch and interdiff

gauravvvv’s picture

StatusFileSize
new349.05 KB

I have applied the patch and it works for me, Adding screenshot for reference. +1 RTBC

gauravvvv’s picture

gauravvvv’s picture

Status: Reviewed & tested by the community » Fixed

Thank you for working on this issue. Committed to 8.x-1.x

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.