Skip to content

Improved UX for feed importer general settings#1112

Merged
Soare-Robert-Daniel merged 5 commits into
developmentfrom
bugfix/pro/884
Aug 7, 2025
Merged

Improved UX for feed importer general settings#1112
Soare-Robert-Daniel merged 5 commits into
developmentfrom
bugfix/pro/884

Conversation

@girishpanchal30
Copy link
Copy Markdown
Contributor

Summary

I have improved the UX for the general settings of the feed importer and added two new options for removing HTML tags and feed import ordering. Also, I have removed the auto-delete images and posts setting as discussed here.

Will affect visual aspect of the product

YES

Screenshots

image image

Check before Pull Request is ready:

Closes:
https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/884
https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/882
https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/862

@girishpanchal30 girishpanchal30 added the pr-checklist-skip Allow this Pull Request to skip checklist. label Jul 30, 2025
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Jul 30, 2025
@pirate-bot
Copy link
Copy Markdown
Contributor

pirate-bot commented Jul 30, 2025

Plugin build for 29c58d3 is ready 🛎️!

Note

You can preview the changes in the Playground

@Soare-Robert-Daniel
Copy link
Copy Markdown
Contributor

@girishpanchal30 you can drop the changes related to fallback image design and functionality since they conflict with the issue https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/885, which has this implementation #1120 (Discussion https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/885#issuecomment-3150255694)

@girishpanchal30
Copy link
Copy Markdown
Contributor Author

@Soare-Robert-Daniel I have reverted the changes related to the fallback image with the latest commit.

Copy link
Copy Markdown
Contributor

@Soare-Robert-Daniel Soare-Robert-Daniel left a comment

Choose a reason for hiding this comment

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

The UI looks ok. I will wait for PHPStan changes, left a mention for it. It is good that we solved some errors, but we should not add new ones.

Comment thread phpstan-baseline.neon Outdated
@Soare-Robert-Daniel Soare-Robert-Daniel merged commit 2bc201f into development Aug 7, 2025
9 checks passed
@Soare-Robert-Daniel Soare-Robert-Daniel deleted the bugfix/pro/884 branch August 7, 2025 14:58
@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.

3 participants