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

fix: Introduce showFloatingDateDivider in MessageListView on an older version too #895

Conversation

rohan20
Copy link

@rohan20 rohan20 commented Feb 10, 2022

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable) [There are no tests]

Description of the pull request

  • Based on the conversation here, my project cannot upgrade to a more recent version of stream_chat_flutter due to a cyclic dependencies issue that I haven't fixed yet. We need this change to fix an issue on stream_chat_flutter: ^2.0.0-nullsafety.2.
  • I used the change from @imtoori as done here originally.

@rohan20
Copy link
Author

rohan20 commented Feb 11, 2022

@xsahil03x Can you please help me with the failing checks? Are they supposed to pass even for a fix on an old version like this one?

@xsahil03x
Copy link
Member

Hey @rohan20 , not really needed it's failing due to melos.
This LGTM. Thanks again for the PR 👍🏼

@xsahil03x
Copy link
Member

Can you also update the version and add a changelog entry for the same? Thanks

@rohan20
Copy link
Author

rohan20 commented Feb 11, 2022

Great. What should the version be? I see that we already have versions 2.0.0-nullsafety.3 till 2.0.0-nullsafety.7.

2.0.0-nullsafety.2-showFloatingDateDivider or something else that you suggest?

image

@xsahil03x
Copy link
Member

@rohan20 let's do 2.0.0-nullsafety.3-beta

@rohan20
Copy link
Author

rohan20 commented Feb 11, 2022

Done! Please merge and create a new tag! Thanks for all your help! ⭐

Copy link
Member

@xsahil03x xsahil03x left a comment

Choose a reason for hiding this comment

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

LGTM

@xsahil03x xsahil03x merged commit cc4ae96 into GetStream:release/2.0.0-nullsafety.2 Feb 11, 2022
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