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

fix(core): label and route fields of useMenu.menuItems shouldn't be deprecated #6353

Conversation

alicanerdurmaz
Copy link
Member

PR Checklist

Please check if your PR fulfills the following requirements:

Bugs / Features

What is the current behavior?

The label and route fields in useMenu().menuItems were marked as deprecated, but they are not deprecated.

What is the new behavior?

This change removes the deprecation warning for these fields because useMenu automatically handles the deprecation of these fields internally.

reproducible example code screenshot

fixes #6352

@alicanerdurmaz alicanerdurmaz requested a review from a team as a code owner September 18, 2024 14:45
Copy link

changeset-bot bot commented Sep 18, 2024

🦋 Changeset detected

Latest commit: 5fabf56

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@refinedev/core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@alicanerdurmaz alicanerdurmaz changed the title Fix/core/label and route fields of menu items should not be deprecated fix(core): label and route fields of useMenu.menuItems shouldn't be deprecated Sep 18, 2024
Copy link

cypress bot commented Sep 18, 2024

refine    Run #11512

Run Properties:  status check passed Passed #11512  •  git commit 658ec5285e ℹ️: Merge 5fabf5621ad3373c5b390fd12452fc40f6c15418 into 6e5d97c2b82aa70f5053bcb70012...
Project refine
Branch Review fix/core/label-and-route-fields-of-menu-items-should-not-be-deprecated
Run status status check passed Passed #11512
Run duration 17m 22s
Commit git commit 658ec5285e ℹ️: Merge 5fabf5621ad3373c5b390fd12452fc40f6c15418 into 6e5d97c2b82aa70f5053bcb70012...
Committer Alican Erdurmaz
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 37
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 383
View all changes introduced in this branch ↗︎

@BatuhanW BatuhanW added this to the October Release milestone Sep 19, 2024
@aliemir aliemir merged commit a0f2d7b into releases/october Sep 23, 2024
89 checks passed
@aliemir aliemir deleted the fix/core/label-and-route-fields-of-menu-items-should-not-be-deprecated branch September 23, 2024 11:58
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.

[BUG] label and route fields of useMenu().menuItems shouldn't be deprecated
3 participants