Skip to content

Picker - migrate prop deprecation end #3737

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 9 commits into
base: v8
Choose a base branch
from

Conversation

adids1221
Copy link
Contributor

@adids1221 adids1221 commented May 20, 2025

Description

Picker - migrate, getItemLabel, getItemLabel prop deprecation end, remove the migrate, getItemLabel, getItemLabel prop from component usage.
Based on #3718

Changelog

Picker - migrate, getItemLabel, getItemLabel prop deprecation end.

Additional info

MADS-4195

Copy link
Collaborator

@M-i-k-e-l M-i-k-e-l left a comment

Choose a reason for hiding this comment

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

@M-i-k-e-l M-i-k-e-l assigned adids1221 and unassigned M-i-k-e-l May 25, 2025
@adids1221 adids1221 requested a review from M-i-k-e-l June 3, 2025 07:02
@adids1221 adids1221 assigned M-i-k-e-l and unassigned adids1221 Jun 3, 2025
@M-i-k-e-l M-i-k-e-l assigned adids1221 and unassigned M-i-k-e-l Jun 3, 2025
@adids1221
Copy link
Contributor Author

@M-i-k-e-l you assigned me without any comment ?

@adids1221 adids1221 assigned M-i-k-e-l and unassigned adids1221 Jun 4, 2025
@@ -73,3 +73,6 @@ Removed (use `PanView` instead)
Use the `backgroundColor` prop instead of `contentContainerStyle={{backgroundColor}}`
Fix card being transparent on Android
`onCollapseChanged` will now be called after the animation has ended (as was intended)

### Picker
Removed `migrate` prop - the picker now only supports the new value format (direct values instead of `{value, label}` objects).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's improve this, see my PM

@M-i-k-e-l
Copy link
Collaborator

@M-i-k-e-l you assigned me without any comment ?

It was pending, look at the PM and improve the docs please

The component was refactored to simplify its API and improve type safety.

Props migration:
- `migrate` - Removed (new API is default)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- `migrate` - Removed (new API is default)
- `migrate` - Removed

There is no old API so there is no default - there is only one API

Comment on lines 82 to 83
- `getItemValue` - Removed (value extraction is now automatic from `item.value`)
- `getItemLabel` - Removed (label extraction is now automatic from `item.label`)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe: "Use item.<X> to get <X>"

@adids1221 adids1221 requested a review from M-i-k-e-l June 10, 2025 08:07
Comment on lines +91 to +92
- `getItemValue` - Removed (use `item.value` to get value)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Search for getItemLabel and getItemValue in the repo and remove them (even in comments)

@M-i-k-e-l M-i-k-e-l assigned adids1221 and unassigned M-i-k-e-l Jun 10, 2025
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