Skip to content

fix: route dashboard submissions through pricing plans - #2999

Open
RabbiIslamRony wants to merge 2 commits into
sovware:developmentfrom
RabbiIslamRony:fix/dashboard-pricing-plan-page
Open

fix: route dashboard submissions through pricing plans#2999
RabbiIslamRony wants to merge 2 commits into
sovware:developmentfrom
RabbiIslamRony:fix/dashboard-pricing-plan-page

Conversation

@RabbiIslamRony

@RabbiIslamRony RabbiIslamRony commented Sep 7, 2026

Copy link
Copy Markdown
Member

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

How to reproduce the issue or how to test the changes

  1. Activate a supported pricing plans extension and configure the Directorist Pricing Plans page.
  2. Sign in as a listing owner and open the Directorist dashboard.
  3. Click Submit Listing and confirm it opens the configured Pricing Plans page.
  4. Disable the pricing plans extension/setting and confirm the same button falls back to the Add Listing page.

The dashboard action previously always used the Add Listing URL, bypassing the configured Pricing Plans page. This change aligns it with the existing Submit Listing widget behavior by using the fee-plan URL only while a fee manager is active.

Verification

  • Focused PHPCS passed for templates/dashboard/nav-buttons.php.
  • PHP syntax check passed.
  • git diff --check passed.
  • Browser QA confirmed:
    • Before: /dashboard//add-listing/
    • After, pricing enabled: /dashboard/ → configured Pricing Plans page
    • Pricing disabled: /dashboard//add-listing/
  • No browser console errors were reported during the verified flow.

GitHub's PHPCS workflow currently stops at actions/checkout before running PHPCS because this fork PR is handled by a trusted pull_request_target workflow that refuses fork code checkout. The focused local PHPCS run itself passes.

Before
https://prnt.sc/6RpfkYRO2Kec
After
https://prnt.sc/wb-qVNCpVecb

Any linked issues

https://team.sovware.com/support/cmtqp5e8t0gpf3ss6ztwu91uf

Checklist

@RabbiIslamRony
RabbiIslamRony changed the base branch from trunk to development September 7, 2026 10:38
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.

2 participants