Problem/Motivation

Module needs to be port to Drupal 9

CommentFileSizeAuthor
#3 3182086-3.patch822 bytesvacho
#2 3182086-2.patch406 byteswadator

Issue fork jsonapi_permission-3182086

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

    jhuret created an issue. See original summary.

    wadator’s picture

    wadator commented
    Status: Active » Needs review
    StatusFileSize
    new 3182086-2.patch406 bytes

    Looks like this module has no deprecated code issues. So add core_version_requirement: ^8 || ^9 to jsonapi_permission.info.yml.

    vacho’s picture

    vacho commented
    StatusFileSize
    new 3182086-3.patch822 bytes

    The patch #2 forget review the module integration con core. Class Drupal\KernelTests\KernelTestBase needs to be use in place to use Drupal\Tests\token\Kernel\KernelTestBase. So I update with this fix.

    • Ruuds committed ac1c3b1 on 8.x-1.x 
      Issue #3182086 by jhuret, watador, vacho: Drupal 9 compatibility and...
    ruuds’s picture

    ruuds commented

    Thank you all!

    ruuds’s picture

    ruuds commented
    Status: Needs review » Fixed

    Status: Fixed » Closed (fixed)

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