Skip to content

feat: added helper notice for headers and proxy#1128

Merged
Soare-Robert-Daniel merged 3 commits into
developmentfrom
feat/helper_proxy
Aug 12, 2025
Merged

feat: added helper notice for headers and proxy#1128
Soare-Robert-Daniel merged 3 commits into
developmentfrom
feat/helper_proxy

Conversation

@RaduCristianPopescu
Copy link
Copy Markdown
Contributor

Summary

Added helper information on proxy & user-agent.

Will affect visual aspect of the product

YES

Screenshots

Headers

CleanShot 2025-08-07 at 16 53 42@2x

Proxy

CleanShot 2025-08-07 at 16 54 07@2x

Other

CleanShot 2025-08-07 at 16 54 24@2x

Closes https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/871

@RaduCristianPopescu RaduCristianPopescu self-assigned this Aug 7, 2025
@RaduCristianPopescu RaduCristianPopescu added the pr-checklist-skip Allow this Pull Request to skip checklist. label Aug 7, 2025
@pirate-bot pirate-bot added pr-checklist-complete The Pull Request checklist is complete. (automatic label) labels Aug 7, 2025
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

This PR adds helpful contextual information to the headers and proxy configuration sections in the Feedzy RSS Feeds plugin settings. The changes provide users with clear guidance on when and why to use these advanced features.

  • Added informational notices with explanations for headers and proxy configurations
  • Included "Learn more" links to external documentation
  • Conditionally hid the generic "Need help?" button when viewing headers or proxy tabs

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
includes/layouts/settings.php Added helper notices with contextual information for headers and proxy sections, and conditionally hide generic help button
css/settings.css Added styling for the new helper notice components with blue theme styling

Comment thread includes/layouts/settings.php Outdated
Comment thread includes/layouts/settings.php Outdated
Comment thread includes/layouts/settings.php Outdated
Comment thread includes/layouts/settings.php Outdated
@pirate-bot
Copy link
Copy Markdown
Contributor

pirate-bot commented Aug 7, 2025

Plugin build for 33535e5 is ready 🛎️!

Note

You can preview the changes in the Playground

Comment thread css/settings.css Outdated
}
}

.feedzy_helper_notice {
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.

Use feedzy-helper-notice

@Soare-Robert-Daniel Soare-Robert-Daniel merged commit 747c2a0 into development Aug 12, 2025
8 checks passed
@Soare-Robert-Daniel Soare-Robert-Daniel deleted the feat/helper_proxy branch August 12, 2025 07:15
@pirate-bot
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Aug 20, 2025
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. released Indicate that an issue has been resolved and released in a particular version of the product.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants