Skip to content

New. UserPassCheck. Debugging functionality using the cron task launch button - #697

Merged
AntonV1211 merged 4 commits into
devfrom
debug_us_pas_chk_av
Aug 3, 2026
Merged

New. UserPassCheck. Debugging functionality using the cron task launch button#697
AntonV1211 merged 4 commits into
devfrom
debug_us_pas_chk_av

Conversation

@AntonV1211

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Debug tab control to schedule the UsersPassCheck cron worker shortly after clicking a button, to help with debugging cron task launch behavior.

Changes:

  • Registers a new debug field debug_user_pass_check in the settings UI.
  • Adds spbc_field_debug_user_pass_check() to render a form that schedules the users_pass_check_worker cron task.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread inc/spbc-settings.php Outdated
Comment thread inc/spbc-settings.php Outdated

@alexandergull alexandergull left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am surprised of a lot of changes required to release a debug feature..

Comment thread js/src/spbc-settings.js
Comment on lines +631 to +636
$st.css('color', 'green').html(
'✓ Done! Total: ' + response.data.total +
', Checked: ' + response.data.checked +
', Leaked: ' + response.data.leaked +
' &mdash; <b>You can now test the login form in incognito.</b>',
);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't an error, but I would write it like this. ``

`&#10003; Done! Total: ${response.data.total},
Checked: ${response.data.checked},
Leaked: ${response.data.leaked}
&mdash; <b>You can now test the login form in incognito.</b>`

@AntonV1211
AntonV1211 merged commit 550a486 into dev Aug 3, 2026
3 checks passed
@AntonV1211
AntonV1211 deleted the debug_us_pas_chk_av branch August 12, 2026 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants