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

CommentFileSizeAuthor
#8 3125893-8.patch417 bytesthepractice
#2 3125893-2.patch349 bytesmo_farhaz

Issue fork trailing_slash-3125893

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

mo_farhaz created an issue. See original summary.

mo_farhaz’s picture

mo_farhaz CreditAttribution: mo_farhaz at Specbee commented
Assigned: mo_farhaz » Unassigned
Status: Active » Needs review
FileSize
3125893-2.patch349 bytes

please review it.

nikolas.tatianenko’s picture

nikolas.tatianenko CreditAttribution: nikolas.tatianenko as a volunteer and at EPAM Systems commented
Assigned: Unassigned » nikolas.tatianenko
Status: Needs review » Reviewed & tested by the community
SAVEL’s picture

SAVEL CreditAttribution: SAVEL as a volunteer and at EPAM Systems commented
Issue tags: +epam-contrib

Thank you, mo_farhaz and Nikolas.

I've also tested the patch.

It's possible to enable Trailing Slash on Drupal 9-beta-3 and I can see slashes are being added. I will merge it to 8.x-1.x

  • SAVEL committed 3448155 on 8.x-1.x authored by mo_farhaz 
    Issue #3125893 by mo_farhaz, nikolas.tatianenko, SAVEL: Drupal 9...
SAVEL’s picture

SAVEL CreditAttribution: SAVEL as a volunteer and at EPAM Systems commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

thepractice’s picture

thepractice CreditAttribution: thepractice at Third and Grove commented
Version: 8.x-1.x-dev » 8.x-1.0
FileSize
3125893-8.patch417 bytes

Attached is patch that applies cleanly to version 8.x-1.0.