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: null check attachment duration #2348

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

tao-qian
Copy link
Contributor

@tao-qian tao-qian commented Dec 15, 2023

🎯 Goal

We saw the following crash
Screenshot 2023-12-15 at 2 25 49 PM

🛠 Implementation details

🎨 UI Changes

iOS
Before After
Android
Before After

🧪 Testing

☑️ Checklist

  • I have signed the Stream CLA (required)
  • PR targets the develop branch
  • Documentation is updated
  • New code is tested in main example apps, including all possible scenarios
    • SampleApp iOS and Android
    • Expo iOS and Android

@khushal87 khushal87 changed the title Fix null check fix: null check attachment duration Dec 21, 2023
@khushal87 khushal87 merged commit 0f516cd into GetStream:develop Dec 21, 2023
3 of 7 checks passed
@github-actions github-actions bot mentioned this pull request Dec 21, 2023
6 tasks
santhoshvai added a commit that referenced this pull request Jan 18, 2024
* fix: channel around message performance issues

* batching data

* fhinal fixes

* restart interval on copy messages state

* remove debug text

* fix test

* fix: message action list item title theme (#2352)

* fix: null check attachment duration (#2348)

* fix: width and height passed to ImageResizer were reversed (#2350)

At least based on the latest documentation, the width should come before height: https://github.com/bamlab/react-native-image-resizer?tab=readme-ov-file#usage-example

* feat: compress images selected through file picker while upload (#2359)

* docs: update example apps README.md with latest steps (#2355)

* docs: update example apps README.md with latest steps

* refactor: delete examples/ExpoMessaging/Readme.md

* refactor: dev setup and sample app docs removed

* docs: add examples directory README.md

* chore: pin vale version

* sample app, fix unread counts

* fix: initial unread scroll

---------

Co-authored-by: Khushal Agarwal <[email protected]>
Co-authored-by: Tao Qian <[email protected]>
@stream-ci-bot
Copy link
Contributor

🎉 This PR is included in version 5.23.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants