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

Render Error for DatePickerInput component: "Check the render method of 'Month'" #392

Closed
kalyncoose opened this issue May 30, 2024 · 6 comments
Labels
needs more info Needs more information needs repro Needs reproducible example stale

Comments

@kalyncoose
Copy link

Issue:
DatePickerInput component starts to render but crashes when attempting to render the Month child component inside the 'Calendar'.

image

Resolution:
We noticed the peerDependencies for react-native-paper is an asterisk * which is a bad idea for ensuring compatibility and letting end users know what version they should be using at a minimum. We upgraded react-native-paper from 5.3.1 to 5.12.3 which resolved the rendering issue - now the DatePickerInput can fully render and load the calendar (albeit pretty slow in a debug mode app). Side note: when upgrading to the new react-native-paper version we had to change an import of Provider to PaperProvider for react-native-paper.

"react-native-paper": "*",

Suggestion:
Please put down actual version numbers instead of asterisks for the peer dependencies of this package.

@iM-GeeKy iM-GeeKy added the needs repro Needs reproducible example label May 31, 2024
Copy link
Contributor

Hey! Thanks for opening the issue. Can you provide a minimal repro which demonstrates the issue? Posting a snippet of your code in the issue is useful, but it's not usually straightforward to run. A repro will help us debug the issue faster. Please try to keep the repro as small as possible. The easiest way to provide a repro is on snack.expo.dev. If it's not possible to repro it on snack.expo.dev, then you can also provide the repro in a GitHub repository.

@chamellion
Copy link

Same issue here

@iM-GeeKy iM-GeeKy added the needs more info Needs more information label Jul 14, 2024
Copy link
Contributor

Hey! Thanks for opening the issue. Can you provide more information about the issue? Please fill the issue template when opening the issue without deleting any section. We need all the information we can, to be able to help. Make sure to at least provide - Current behaviour, Expected behaviour, A way to reproduce the issue with minimal code (link to snack.expo.dev) or a repo on GitHub, and the information about your environment (such as the platform of the device, versions of all the packages etc.).

@amjadbouhouch
Copy link

+1

@onebigdoor
Copy link

i got this error and resolved it by updating to the latest version of react-native-paper (note that it's a separate package from react-native-paper-dates)

Copy link
Contributor

Hello 👋, this issue has been open for more than a month without a repro or any activity. If the issue is still present in the latest version, please provide a repro or leave a comment within 7 days to keep it open, otherwise it will be closed automatically. If you found a solution or workaround for the issue, please comment here for others to find. If this issue is critical for you, please consider sending a pull request to fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs more info Needs more information needs repro Needs reproducible example stale
Projects
None yet
Development

No branches or pull requests

5 participants