Problem/Motivation

Many sites probably will want some custom paths that this module can ignore. For example, Drupal Commerce sites might want to sign up users during checkout process, and want users to fill out more information when the checkout is complete. Therefore, the site want the URLs for checkout to be excluded (or to be on the whitelist).

Steps to reproduce

Proposed resolution

Add a whitelist support. Site admin can input a URL per line, like Block's visibility settings

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 3259820-2.patch6.18 KBskyredwang
#2 7d9076ce-5053-4378-83bf-a090649d51a3.jpeg133.7 KBskyredwang

Comments

skyredwang created an issue. See original summary.

skyredwang’s picture

skyredwang
Location Shanghai
commented
Status: Active » Needs review
StatusFileSize
new 7d9076ce-5053-4378-83bf-a090649d51a3.jpeg133.7 KB
new 3259820-2.patch6.18 KB

Here is a patch. The admin UI now has a new config, see screenshot below:

pages to not enforce

Status: Needs review » Needs work

The last submitted patch, 2: 3259820-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

skyredwang’s picture

skyredwang
Location Shanghai
commented
Status: Needs work » Needs review

The failing standard is from the existing code not from the patch

cosolom’s picture

cosolom commented
Status: Needs review » Fixed

Thank you, commited

skyredwang’s picture

skyredwang
Location Shanghai
commented

Can you please merge 8.x-1.6 back to 8.x-1.x? So, the head stays as head

cosolom’s picture

cosolom commented

Done

Status: Fixed » Closed (fixed)

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

joekers’s picture

joekers
Primary language English
commented

Just wanted to comment in case it helps anyone else out - when using this module with the Session Limit module, the '/session-limit' path can be added so that they work together.