Looks like the module has no deprecated code usages, so we just need to mark it as compatible with D9.

So, add core_version_requirement: ^8 || ^9 to all info.yml file as per the definition https://www.drupal.org/node/3070687.

CommentFileSizeAuthor
#8 Drupal-9-readiness-13576005-8.patch373 byteskishor_kolekar
#4 Screenshot 2020-02-05 at 4.28.47 PM.png119.65 KBRangaswini
#4 disable_back_button_deprecated_code-3103146-4.patch373 bytesRangaswini
#2 updated-info-file.patch398 bytessahana _n

Comments

Sahana _N created an issue. See original summary.

sahana _n’s picture

sahana _n commented
Status: Active » Needs review
StatusFileSize
new updated-info-file.patch398 bytes

Please review the patch.

Rangaswini’s picture

Rangaswini commented
Assigned: Unassigned » Rangaswini
Status: Needs review » Needs work
Rangaswini’s picture

Rangaswini commented
Assigned: Rangaswini » Unassigned
Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new disable_back_button_deprecated_code-3103146-4.patch373 bytes
new Screenshot 2020-02-05 at 4.28.47 PM.png119.65 KB
naresh_bavaskar’s picture

naresh_bavaskar
Location Pune, Maharashtra
commented
Assigned: Unassigned » naresh_bavaskar
naresh_bavaskar’s picture

naresh_bavaskar
Location Pune, Maharashtra
commented
Assigned: naresh_bavaskar » Unassigned
Status: Needs review » Reviewed & tested by the community

patch looks fine, all files code having drupal 9 compatible.

kishor_kolekar’s picture

kishor_kolekar commented
Issue tags: +Drupal 9 porting dayDrupal 9 readinessDrupal 9 compatibility
kishor_kolekar’s picture

kishor_kolekar commented
Status: Reviewed & tested by the community » Needs work
Issue tags: -Drupal 9 porting dayDrupal 9 readinessDrupal 9 compatibility +Drupal 9 porting day, +Drupal 9 readiness, +Drupal 9 compatibility
StatusFileSize
new Drupal-9-readiness-13576005-8.patch373 bytes

Patch name should follow the pattern [description]-[issue-number]-[comment-number].patch

Fixing the patch name!!

hardik_patel_12’s picture

hardik_patel_12 commented
Status: Needs work » Fixed

@thank you all , patch at #4 is has been pushed to 8.x-1.x branch.

hardik_patel_12’s picture

hardik_patel_12 commented

@kishor_kolekar , thanks for your patch but i think patch at #4 is already done same thing.

kishor_kolekar’s picture

kishor_kolekar commented

sorry I missed it @Hardik_Patel_12

Status: Fixed » Closed (fixed)

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

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented
Issue tags: -Drupal 9 readiness

Removing outdated tag.