Skip to content

Conversation

@masteradhoc
Copy link
Contributor

All Submissions:

Changes proposed in this Pull Request:

In the Plugins Admin Screen this PR adds a link for users to easily access the wc-smooth-generator settings.

Closes #169 .

How to test the changes in this Pull Request:

  1. apply the pr
  2. visit /wp-admin/plugins.php and see if there is a setting
image

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • [] Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?

Changelog entry

Add a settings Link in plugin overview

Enter a summary of all changes on this Pull Request. This will appear in the changelog if accepted.

FOR PR REVIEWER ONLY:

  • I have reviewed that everything is sanitized/escaped appropriately for any SQL or XSS injection possibilities. I made sure Linting is not ignored or disabled.

Copy link
Contributor

@layoutd layoutd left a comment

Choose a reason for hiding this comment

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

Looks good, just a couple changes!

Copy link
Contributor

@layoutd layoutd left a comment

Choose a reason for hiding this comment

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

Looks good, couple of changes

masteradhoc and others added 2 commits October 7, 2025 18:58
Co-authored-by: Justin P <228780+layoutd@users.noreply.github.com>
Co-authored-by: Justin P <228780+layoutd@users.noreply.github.com>
@masteradhoc
Copy link
Contributor Author

Thanks @layoutd - commit suggestions were added :)

@layoutd
Copy link
Contributor

layoutd commented Oct 8, 2025

Great, thanks @masteradhoc!

@layoutd layoutd merged commit 0ee6930 into woocommerce:trunk Oct 8, 2025
@masteradhoc masteradhoc deleted the patch-6 branch October 8, 2025 08:30
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.

[Enhancement]: Make Settings more discoverable

2 participants