react-admin icon indicating copy to clipboard operation
react-admin copied to clipboard

Remove deprecated props

Open adguernier opened this issue 1 year ago • 1 comments

Problem

Some props are deprecated

Solution

Remove them

TODO

Remove:

  • [x] disableNotification from useCheckAuth
  • [x] disableNotification from useLogoutIfAccessDenied
  • [x] ~resource from useDeleteWithConfirmController~
  • [x] ~resource from useDeleteWithUndoController~
  • [x] data from CreateControllerResult
  • [x] referenceSource from useReferenceInputController
  • [ ] saving from SaveContextValue ? :thinking:
  • [ ] useGetValidationErrorMessage ? :thinking:
    • [ ] <ArrayInput> has to be adapted
  • [x] ~useLocale~
  • [x] ~useSetLocale~
  • [x] linkToRecord
  • [x] resolveRedirectTo
  • [x] onError type from ra-core type.ts
  • [x] formClassName from FieldProps
  • [ ] fullWidth from FieldProps
    • [ ] many occurences
  • [x] PublicFieldProps from FieldProps
  • [x] InjectedFieldProps from FieldProps
  • [x] formClassName from CommonInputProps
  • [ ] sidebarIsOpen from DashboardMenuItem
    • [ ] check the impact in ra-no-code before
  • [ ] sidebarIsOpen from MenuItemLink
    • [ ] check the impact in ra-no-code before
  • [ ] rowStyle from Datagrid
  • [x] PaginationLimit
  • [x] rowStyle from SimpleList
  • [x] theme from ThemeProvider

adguernier avatar Apr 22 '24 15:04 adguernier

I think we can de-deprecate and keep in v5 the following hooks

  • useLocale
  • useSetLocale

fzaninotto avatar Apr 25 '24 16:04 fzaninotto