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

refactor!: remove deprecated ios stack animation #2361

Merged

Conversation

maciekstosio
Copy link
Contributor

@maciekstosio maciekstosio commented Sep 30, 2024

@maciekstosio maciekstosio marked this pull request as ready for review September 30, 2024 15:59
satya164 pushed a commit to react-navigation/react-navigation that referenced this pull request Oct 1, 2024
This PR aligns JSDoc for "stackAnimation" with current state of
react-native-screens (4.0.0)
software-mansion/react-native-screens#2361.
@maciekstosio maciekstosio requested review from tboba and kkafar and removed request for tboba October 1, 2024 08:27
@kkafar kkafar changed the title chore: remove deprecated ios stack animation refactor!: remove deprecated ios stack animation Oct 2, 2024
Copy link
Member

@kkafar kkafar left a comment

Choose a reason for hiding this comment

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

This seems good. Thank you!

@maciekstosio maciekstosio changed the title refactor!: remove deprecated ios stack animation refactor: remove deprecated ios stack animation Oct 2, 2024
@maciekstosio maciekstosio changed the title refactor: remove deprecated ios stack animation refactor!: remove deprecated ios stack animation Oct 2, 2024
@maciekstosio maciekstosio merged commit c711d3c into main Oct 2, 2024
9 checks passed
@maciekstosio maciekstosio deleted the @maciekstosio/Remove-deprecated-ios-stackAnimation branch October 2, 2024 13:55
ja1ns pushed a commit to WiseOwlTech/react-native-screens that referenced this pull request Oct 9, 2024
## Description

Removes stackAnimation "ios" option in v4 after making it deprecated in
3.x.


## Checklist

- [ ] Included code example that can be used to test this change
- [x] Updated TS types
- [x] Updated documentation: <!-- For adding new props to native-stack
-->
- [x]
https://github.com/software-mansion/react-native-screens/blob/main/guides/GUIDE_FOR_LIBRARY_AUTHORS.md
- [x]
https://github.com/software-mansion/react-native-screens/blob/main/native-stack/README.md
- [x]
https://github.com/software-mansion/react-native-screens/blob/main/src/types.tsx
- [x]
https://github.com/software-mansion/react-native-screens/blob/main/src/native-stack/types.tsx
- [ ] Ensured that CI passes
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