UI update. Renamed the account creation button and description. Add new params in request - #698
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the signup wizard UI text and tags “auto key” registrations that originate from the React setup wizard so they can be identified downstream (via the CleanTalk API request payload).
Changes:
- Added a
lead_sourceparameter to the CleanTalkget_api_keyAPI wrapper and passed it from the auto-key AJAX handler when invoked from the React wizard. - Updated the React signup wizard CTA text (“Activate Security”) and included a
spbct_wizard_request=1marker in the AJAX request. - Regenerated the public React bundle to reflect the UI/AJAX changes and updated webpack runtime output.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| lib/CleantalkSP/Common/API.php | Extends method__get_api_key() request payload with lead_source. |
| inc/spbc-settings.php | Detects wizard-originated AJAX calls and passes lead_source into API key retrieval. |
| js/src/react/components/SignupWizard/Screens/SignupInterface.js | Updates wizard button text and adds spbct_wizard_request flag to AJAX payload. |
| js/public/spbct-react-bundle.js | Rebuilt bundle containing the UI/AJAX updates (and runtime output changes). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| /******/ // define __esModule on exports | ||
| /******/ __webpack_require__.r = (exports) => { | ||
| /******/ if(typeof Symbol !== 'undefined' && Symbol.toStringTag) { | ||
| /******/ if(Symbol.toStringTag) { |
Contributor
Author
There was a problem hiding this comment.
I disagree
alexander-b-clean
changed the title
alexander-b-clean
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
Suppressed comments (1)
inc/spbc-settings.php:3934
- The inline comment refers to a “Connect” button, but the UI text in the React signup wizard was renamed to “Activate Security”. Updating the comment will keep the code documentation consistent with the current UI and reduce confusion during future maintenance.
// Mark the registrations made by the Connect button of the setup wizard
alexandergull
requested changes
Aug 3, 2026
alexandergull
approved these changes
Aug 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://app.doboard.com/1/task/53201