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

CommentFileSizeAuthor
#5 drupal_check.png16.26 KBadityasingh
#5 apply_patch.png26.25 KBadityasingh
#3 3133731-3.patch357 bytesmohrerao

Issue fork starrating-3133731

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

Comments

nitvirus created an issue. See original summary.

nitvirus’s picture

nitvirus commented
Issue summary: View changes
mohrerao’s picture

mohrerao commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 3133731-3.patch357 bytes

Adding a patch.

adityasingh’s picture

adityasingh commented
Assigned: nitvirus » adityasingh

I'm reviewing this

adityasingh’s picture

adityasingh commented
Assigned: adityasingh » Unassigned
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new apply_patch.png26.25 KB
new drupal_check.png16.26 KB

After applying the patch, i have checked drupal 9 compatibility and found no errors.
I am setting the status RTBC.

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

Jelle_S opened merge request !1

rossidrup’s picture

rossidrup commented

how to install it?
I ran composer require 'drupal/starrating:^4.0@alpha' and it did not work

Deno’s picture

Deno commented

This was a year ago. Why don't you declare the module (well aloha version thereof) D9 ready?

rossidrup’s picture

rossidrup commented

how? with patch?

karthikeyan-manivasagam’s picture

karthikeyan-manivasagam commented

Sure thanks everyone will apply & release in next version soon

karthikeyan-manivasagam’s picture

karthikeyan-manivasagam commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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