Skip to content

Conversation

RaduCristianPopescu
Copy link
Contributor

@RaduCristianPopescu RaduCristianPopescu commented Aug 5, 2025

Summary

Refactors the dashboard/support page functionality by extracting the license from the settings page into a dedicated tab for better code organization and maintainability.

For license, the change is in the PRO: https://github.com/Codeinwp/feedzy-rss-feeds-pro/pull/924

Compatiblity

Newer versions will use the new License tab, while the old versions will use the old location (if an old feedzy lite or pro version is present, the old location will be used)

Will affect visual aspect of the product

YES

Screenshots

Recording.6.mp4

Test instructions

  • Check the new tabs
  • Check compatibility between versions

Check before Pull Request is ready:

Check reliability

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

@RaduCristianPopescu RaduCristianPopescu self-assigned this Aug 5, 2025
@RaduCristianPopescu RaduCristianPopescu added the pr-checklist-skip Allow this Pull Request to skip checklist. label Aug 5, 2025
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Aug 5, 2025
@pirate-bot
Copy link
Contributor

pirate-bot commented Aug 5, 2025

Plugin build for d2177f7 is ready 🛎️!

Note

You can preview the changes in the Playground

Copy link

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

Refactors the dashboard/support page by extracting license functionality from the settings page into a dedicated tab, reorganizing the navigation structure and improving code maintainability.

  • Extracts license key validation logic from settings JavaScript into a dedicated license module
  • Reorganizes admin menu structure with Dashboard as the primary entry point
  • Adds license management as a separate tab within the Dashboard page

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
js/feedzy-setting.js Removes license key handling code that was moved to dedicated file
js/feedzy-license.js New dedicated module for license key validation with improved structure
includes/layouts/setup-wizard.php Updates redirect URL to point to dashboard instead of imports page
includes/layouts/header.php Changes page title from "Support" to "Dashboard"
includes/layouts/feedzy-support.php Adds license tab and navigation links to settings/integration pages
includes/layouts/feedzy-license.php New template file for license tab content
includes/feedzy-rss-feeds.php Adjusts menu hook priorities and adds RSS to Social menu action
includes/admin/feedzy-rss-feeds-admin.php Major reorganization of menu structure and script enqueuing logic

@Soare-Robert-Daniel Soare-Robert-Daniel changed the title Refactor/dashboard page refactor: transform the Support page into the Dashboard Aug 12, 2025
@poonam279
Copy link

@Soare-Robert-Daniel, If we skip the setup wizard or choose the shortcode/block option, there’s no way to access the import option. I selected the shortcode option, and the draft page was created as expected, but the setup wizard option still appears, and the import option is missing from the menu.

Markup on 2025-08-12 at 17:43:15

@Soare-Robert-Daniel Soare-Robert-Daniel merged commit 422a861 into development Aug 13, 2025
10 of 12 checks passed
@Soare-Robert-Daniel Soare-Robert-Daniel deleted the refactor/dashboard_page branch August 13, 2025 11:30
@pirate-bot
Copy link
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