Skip to content

Remove deprecated partial#6431

Open
BrentWheeldon wants to merge 1 commit intosolidusio:mainfrom
BrentWheeldon:main
Open

Remove deprecated partial#6431
BrentWheeldon wants to merge 1 commit intosolidusio:mainfrom
BrentWheeldon:main

Conversation

@BrentWheeldon
Copy link
Copy Markdown

Summary

The menu item still renders as expected in the DOM, just without the DEPRECATION WARNING: Using the "spree/admin/shared/_promotion_sub_menu" partial is deprecated, please use MenuItem#children instead. deprecation warning in the logs.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed:

  • 📖 I have updated the README to account for my changes.
  • 📑 I have documented new code with YARD.
  • 🛣️ I have opened a PR to update the guides.
  • ✅ I have added automated tests to cover my changes.
  • 📸 I have attached screenshots to demo visual changes.

The menu item still renders as expected in the DOM, just without the
`DEPRECATION WARNING: Using the "spree/admin/shared/_promotion_sub_menu"
partial is deprecated, please use MenuItem#children instead.`
deprecation warning in the logs.
@BrentWheeldon BrentWheeldon requested a review from a team as a code owner April 2, 2026 18:18
@github-actions github-actions bot added the changelog:solidus_legacy_promotions Changes to the solidus_legacy_promotions gem label Apr 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:solidus_legacy_promotions Changes to the solidus_legacy_promotions gem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant