Suggestion for improved prompt in configuration: Activating for form ids

Created on 20 January 2022, over 2 years ago
Updated 27 January 2023, over 1 year ago

Problem/Motivation

I had trouble trying to activate the module for the form/page where users update their passwords eg example.com/user/666/edit.
Eventually I tracked the issue down to my having a space after the comma, eg "user_login_form, user_form".

Steps to reproduce

In the module configuration, enter multiple form ids with a comma and space between them.
The module will not activate for the desired second form.

Proposed resolution

While this was obviously my error, a change to the prompt for the field may have seen me fix this earlier, perhaps "Please enter the form id(s) by separating it with a comma. Ensure there are no spaces between form ids."

It could also be helpful to provide common form ids in this prompt:
Other common forms are "user_form" for when users are changing their password, and "user_register_form" for setting password while registering.
These would help make a simple and useful module even easier to set up!

Remaining tasks

User interface changes

API changes

Data model changes

✨ Feature request
Status

Fixed

Version

6.0

Component

User interface

Created by

πŸ‡³πŸ‡ΏNew Zealand agt

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024