Checked the Deprecated code report using drupal-check and upgrade status found:

[OK] No errors

We just need to add core_version_requirement: ^8 || ^9 to .info.yml file and the module is ready for D9.

CommentFileSizeAuthor
#2 D9-3138939-2.patch370 bytesdishabhadra

Issue fork views_simplechart-3138939

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dishabhadra created an issue. See original summary.

dishabhadra’s picture

dishabhadra CreditAttribution: dishabhadra at Axelerant commented
Assigned: dishabhadra » Unassigned
Status: Active » Needs review
FileSize
D9-3138939-2.patch370 bytes

Applied the patch.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Issue tags: -Drupal 9 compatibility Drupal 9 porting weekend DIACWMay2020 +Drupal 9 compatibility, +Drupal 9 porting weekend, +DIACWMay2020

Fixing tags.

jungle’s picture

jungle
He/Him
Location Chongqing, China
CreditAttribution: jungle as a volunteer commented
Version: 8.x-1.2 » 8.x-1.x-dev

Let's change it to a dev branch.

JayKandari’s picture

JayKandari
Primary language Hindi
CreditAttribution: JayKandari as a volunteer and at QED42 for Drupal India Association commented
Status: Needs review » Reviewed & tested by the community

Thanks @dishabhadra for the patch.

#2 applies cleanly.

Upgrade Status doesn't report any issues:

Ran Rector : No issues reported.

❯ vendor/bin/rector process modules/custom/views_simplechart --dry-run
Rector 0.8.x-dev@80f429b
Config file: rector.yml

 9/9 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

                                                                                                                        
 [OK] Rector is done!

Marking it as RTBC. :) Thanks

credevator made their first commit to this issue’s fork.

_shY made their first commit to this issue’s fork.

_shY opened merge request !1

  • _shY committed b8b020ad on 8.x-1.x 
    Issue #3138939: D9 Readiness
_shY’s picture

_shY
Primary language Ukrainian
Location Ukraine, Lutsk 🇺🇦
CreditAttribution: _shY at DevBranch, Drupal Ukraine Community commented
Status: Reviewed & tested by the community » Fixed

Merged to 8.x-1.x branch.
Thanks, everyone for your time.

Status: Fixed » Closed (fixed)

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