Skip to content

refactor: transform the Support page into the Dashboard #1122

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: development
Choose a base branch
from

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

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.

4 participants