The module has a config form but doesn't provide schema config for that.

CommentFileSizeAuthor
#2 3403058-2.patch589 byteskeshavv

Issue fork simpleautologout-3403058

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

AstonVictor created an issue. See original summary.

keshavv’s picture

keshavv commented
Status: Active » Needs review
StatusFileSize
new 3403058-2.patch589 bytes

I have created patch please review.

AstonVictor opened merge request !6

astonvictor’s picture

astonvictor commented
Assigned: astonvictor » Unassigned

Hi @keshavv,
your schema is not correct. e.g. you added timeout_refresh_rate as type: sequence but it's a simple text with only one value.
Also, using patches is an old way. All changes should be added via MRs.

Created a new MR with a schema config. tested it with a Configuration Inspector module.

astonvictor’s picture

astonvictor commented
keshavv’s picture

keshavv commented

Hi @AstonVictor
Just saw that the timeout_refresh_rate is a select field, not a text field.
Please correct me if I am wrong.
https://git.drupalcode.org/project/simpleautologout/-/blob/2.x-dev/src/F...

astonvictor’s picture

astonvictor commented

Hi @keshavv,
I didn't mean the field type, I meant that it contains a simple text and only one value.
if you look at simpleautologout.settings.yml config you'll find e.g. timeout_refresh_rate: '240' line.

astonvictor’s picture

astonvictor commented
Status: Needs review » Fixed

The MR was merged.

Status: Fixed » Closed (fixed)

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