Skip to content
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

Set the right menu for reviews #2680

Merged
merged 2 commits into from
Nov 16, 2022
Merged

Set the right menu for reviews #2680

merged 2 commits into from
Nov 16, 2022

Conversation

luigifab
Copy link
Contributor

@luigifab luigifab commented Nov 6, 2022

Description

There are many "mistakes" for _setActiveMenu. This PR set the right menu for reviews and ratings.

OpenMage 20.0.16 / PHP 8.0.25

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All automated tests passed successfully (all builds are green)

@github-actions github-actions bot added Component: Adminhtml Relates to Mage_Adminhtml Component: Review Relates to Mage_Review labels Nov 6, 2022
</pending>
<all translate="title" module="review">
<title>All Reviews</title>
<action>adminhtml/catalog_product_review/</action>
<action>adminhtml/catalog_product_review/index</action>
Copy link
Contributor

Choose a reason for hiding this comment

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

Why change this? It'll default to index.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

only to not have a ending slash

Co-authored-by: Ng Kiat Siong <kiatsiong.ng@gmail.com>
@fballiano fballiano merged commit 3f83930 into OpenMage:1.9.4.x Nov 16, 2022
@github-actions
Copy link
Contributor

Unit Test Results

1 files  ±0  1 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
7 runs  ±0  5 ✔️ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 3f83930. ± Comparison against base commit 62d2598.

@luigifab luigifab deleted the review-menu branch November 16, 2022 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Adminhtml Relates to Mage_Adminhtml Component: Review Relates to Mage_Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants