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

[Autocomplete] Provide prop to disable option equal to value warning #36275

Conversation

grzegorz-jazurek
Copy link

PR contains:

@mui-bot
Copy link

mui-bot commented Feb 20, 2023

Netlify deploy preview

https://deploy-preview-36275--material-ui.netlify.app/

Bundle size report

Details of bundle changes

Generated by 🚫 dangerJS against 57d0a02

@zannager zannager added the component: autocomplete This is the name of the generic UI component, not the React module! label Feb 21, 2023
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Feb 21, 2023
@grzegorz-jazurek grzegorz-jazurek force-pushed the bugfix/disable-option-equal-to-value-warning branch from 689e351 to 57d0a02 Compare February 22, 2023 11:59
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Feb 22, 2023
@grzegorz-jazurek
Copy link
Author

@michaldudak I update the branch to resolve conflicts

@ZeeshanTamboli ZeeshanTamboli changed the title Bugfix/disable option equal to value warning [Autocomplete] Provide prop to disable option equal to value warning Feb 26, 2023
Copy link
Member

@ZeeshanTamboli ZeeshanTamboli left a comment

Choose a reason for hiding this comment

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

My concern with having this prop is that developers may misuse it where it is expected to fix the code based on the warning.

cc @michaldudak

@ZeeshanTamboli ZeeshanTamboli added bug 🐛 Something doesn't work and removed bug 🐛 Something doesn't work labels May 18, 2023
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jun 12, 2023
@drewcorlin1
Copy link

Is there any chance of resurrecting this MR? I find myself in a similar situation to many of the comments on #29727. @ZeeshanTamboli could you explain a scenario you're thinking of, I might just be missing something, but I see valid cases for disabling this warning.

@michaldudak
Copy link
Member

See #29727 (comment)

@mj12albert
Copy link
Member

I will close this as we have decided to remove the warning #29727 (comment)

@mj12albert mj12albert closed this Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: autocomplete This is the name of the generic UI component, not the React module! PR: out-of-date The pull request has merge conflicts and can't be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants