Comments

jayasree.e created an issue. See original summary.

jayasree.e’s picture

Assigned: jayasree.e » Unassigned
Status: Active » Needs review
StatusFileSize
new435 bytes

Add core_version_requirement: ^8 || ^9. Please review.

gaëlg’s picture

Status: Needs review » Needs work

According to drupal-check, the module has no deprecation problem. So, indeed, we only need to update .info.yml file. But also in submodules like varnish_purge_tags.

gaëlg’s picture

Issue tags: +Drupal 9 compatibility
sja112’s picture

Status: Needs work » Needs review
StatusFileSize
new2.07 KB
new1.65 KB

Updated .info.yml file of module and submodules.

sja112’s picture

StatusFileSize
new2.23 KB
sja112’s picture

StatusFileSize
new2.32 KB
sja112’s picture

nitesh624’s picture

Assigned: Unassigned » nitesh624
Status: Needs review » Needs work
nitesh624’s picture

StatusFileSize
new2.07 KB
new1.77 KB
nitesh624’s picture

StatusFileSize
new2.23 KB
new1.08 KB
nitesh624’s picture

StatusFileSize
new2.39 KB

update the composer.json and revert th .info.yml

nitesh624’s picture

StatusFileSize
new2.07 KB
new264 bytes

Composer require failed because of https://www.drupal.org/project/varnish_purge/issues/2845054.
however it applied successfully on my local machine

nitesh624’s picture

StatusFileSize
new98.89 KB

composer

nitesh624’s picture

Status: Needs work » Needs review
aspilicious’s picture

Status: Needs review » Needs work

You need to remove the "core" line from the info file.

nitesh624’s picture

StatusFileSize
new2.12 KB
new1.87 KB

Thanks @aspilicious

nitesh624’s picture

nitesh624’s picture

StatusFileSize
new1.87 KB

Sorry for the wrong extension of interdiff file in #17

aspilicious’s picture

Status: Needs work » Reviewed & tested by the community

This is OK.

sja112’s picture

Assigned: nitesh624 » Unassigned
tobybellwood’s picture

Hi,

Just checking to see if there is an aim to get this merged (and hopefully released?) prior to the release of D9 next week, and if there's anything we can do to help out?

We'd obviously prefer not to push out a Drupal9 version of our example project without varnish purging support.

Thanks for your hard work to date!

  • nitesh624 authored 0fbb754 on 9.x-1.x
    Issue #3124349 by nitesh624, sja112, jayasree.e: Drupal 9 Readiness
    

  • nitesh624 authored 0fbb754 on 8.x-2.x
    Issue #3124349 by nitesh624, sja112, jayasree.e: Drupal 9 Readiness
    
littlethoughts’s picture

Version: 8.x-1.x-dev » 8.x-2.x-dev
Status: Reviewed & tested by the community » Fixed

Thanks a lot, included in the new 8.2 release.

naveenvalecha’s picture

Version: 8.x-2.x-dev » 8.x-1.x-dev
Status: Fixed » Patch (to be ported)
Issue tags: +backport

@littlethouhghts,
Any plans to release a new 8.x-1.x release with D9 compatibility?

naveenvalecha’s picture

Status: Patch (to be ported) » Fixed

Fixing the issue status.

idebr’s picture

@littlethoughts What is the reasoning to create a new major branch? It seems 8.x-1.x and 8.x-2.x are perfectly compatible.

Status: Fixed » Closed (fixed)

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