Enhanced integrations settings tab#588
Conversation
11eeed3 to
7add47f
Compare
|
Tip You can preview the changes in the Playground Plugin build for 1a347d5 is ready 🛎️!
|
There was a problem hiding this comment.
Pull request overview
Enhances the Integrations settings tab UX (REST API + WCFM) by adding richer section headers, REST API helper UI (base URL + copy), a password-style secret key field with reveal/copy controls, and an “available endpoints” reference block.
Changes:
- Add REST API Base URL (readonly + copy) and switch REST secret key to a password-style input with reveal/copy controls.
- Add an “Available Endpoints” template + styling to document REST routes and link to the full API guide.
- Add section header icons/status badges and supporting JS/CSS (copy feedback, badge toggling, eye toggle).
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
phpstan-baseline.neon |
Adjusts baseline entries related to admin settings template analysis. |
backend/templates/inputs/text.php |
Adds support for readonly + custom class on text inputs (used by Base URL). |
backend/templates/inputs/password.php |
New password input template (secret key reveal/copy UI). |
backend/templates/available-endpoints.php |
New template rendering REST endpoint list + API guide link. |
backend/templates/admin-settings.php |
Integrations UI enhancements: section icons/badges, readonly copy block, custom endpoints rendering. |
backend/settings-panel.class.php |
Adds localized strings for badge/copy feedback (Active/Inactive/Copied). |
backend/options.php |
Adds Base URL field, changes secret key to password, adds “available endpoints” entry. |
backend/assets/settings.js |
Implements reveal/copy handlers and live badge updates. |
backend/assets/settings.css |
Styles for section heading, badges, endpoint list, password/copy controls. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
For the design, I think for the endpoint presentation, we can have some special section that looks much better. Currently; the endpoints description is listed like a setting options. Here is a sort of design reference that can work with the current layout (I am not referring to be 1-1 but to some clear separation on things)
@poonam279 @vytisbulkevicius if you some feedback for the design in the current implementation of the PR. |
|
@Soare-Robert-Daniel yes, your suggestion looks better to me and more clear. Small thing is probably for 2nd Toggle (WCFM Vendors) we can also use "Enable" instead of 'Yes" to keep consistency. |
|
@Soare-Robert-Daniel I have improved the UI as you suggested. Please check it and let me know if any changes are needed. Thanks! |
|
@girishpanchal30, The layout looks good. Could you please update the “Full API Guide” link? It’s currently not working. Also, a documentation is available, so I think it would be better to link to that instead of the PDF. Here’s the correct link: |
|
@poonam279 I have updated the docs link with the latest commit. Thanks! |

Screenshot
Check before Pull Request is ready:
Closes https://github.com/Codeinwp/ppom-pro/issues/686