Skip to content

Enhanced integrations settings tab#588

Merged
Soare-Robert-Daniel merged 4 commits intodevelopmentfrom
feat/pro/686
Apr 30, 2026
Merged

Enhanced integrations settings tab#588
Soare-Robert-Daniel merged 4 commits intodevelopmentfrom
feat/pro/686

Conversation

@girishpanchal30
Copy link
Copy Markdown
Contributor

@girishpanchal30 girishpanchal30 commented Apr 24, 2026

Screenshot

image

Check before Pull Request is ready:

Closes https://github.com/Codeinwp/ppom-pro/issues/686

@girishpanchal30 girishpanchal30 requested a review from Copilot April 24, 2026 10:07
@girishpanchal30 girishpanchal30 added the pr-checklist-skip Allow this Pull Request to skip checklist. label Apr 24, 2026
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Apr 24, 2026
@pirate-bot
Copy link
Copy Markdown
Contributor

pirate-bot commented Apr 24, 2026

Tip

You can preview the changes in the Playground

Plugin build for 1a347d5 is ready 🛎️!

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

Comment thread backend/assets/settings.css Outdated
Comment thread backend/assets/settings.js Outdated
Comment thread backend/templates/admin-settings.php Outdated
Comment thread backend/templates/admin-settings.php Outdated
Comment thread backend/templates/admin-settings.php Outdated
Comment thread backend/templates/admin-settings.php Outdated
Comment thread backend/templates/available-endpoints.php Outdated
Comment thread backend/options.php Outdated
Comment thread backend/templates/admin-settings.php
Comment thread backend/templates/available-endpoints.php Outdated
Comment thread backend/assets/settings.js
Comment thread backend/templates/available-endpoints.php
@Soare-Robert-Daniel
Copy link
Copy Markdown
Contributor

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)

image

@poonam279 @vytisbulkevicius if you some feedback for the design in the current implementation of the PR.

@vytisbulkevicius
Copy link
Copy Markdown
Contributor

@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.

@girishpanchal30
Copy link
Copy Markdown
Contributor Author

@Soare-Robert-Daniel I have improved the UI as you suggested. Please check it and let me know if any changes are needed. Thanks!

@poonam279
Copy link
Copy Markdown

@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:
https://docs.themeisle.com/product-option-manager/ppom-api-documentation

@girishpanchal30
Copy link
Copy Markdown
Contributor Author

@poonam279 I have updated the docs link with the latest commit. Thanks!

@Soare-Robert-Daniel Soare-Robert-Daniel merged commit e393df7 into development Apr 30, 2026
10 checks passed
@Soare-Robert-Daniel Soare-Robert-Daniel deleted the feat/pro/686 branch April 30, 2026 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants