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: multiple mentions render in message text #2286

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

khushal87
Copy link
Member

🎯 Goal

Fixes #2283

The PR also updates the dependencies of Axios in packages and sample apps after #2281

🛠 Implementation details

The regex is used in individual usernames rather than the accumulator of the reduce method.

🎨 UI Changes

Before:
Simulator Screenshot - iPhone SE (3rd generation) - 2023-11-06 at 13 26 19
After:
Simulator Screenshot - iPhone SE (3rd generation) - 2023-11-06 at 13 26 34

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

@santhoshvai santhoshvai merged commit 97e9c46 into develop Nov 6, 2023
6 checks passed
@santhoshvai santhoshvai deleted the khushal87/fix-mentioned-users branch November 6, 2023 09:25
@github-actions github-actions bot mentioned this pull request Nov 6, 2023
khushal87 added a commit that referenced this pull request Nov 6, 2023
* feat: apply theme to SendButton internal icons (#2280)

* fix: crash when opening null url (#2134)

* fix: crash when opening null url

* fix: only check url if type is video
previous if would fail the validation for non-video attachments

---------

Co-authored-by: Arthur Geron <[email protected]>

* feat: upgrade axios to v1 (#2281)

* fix: multiple mentions render in message text (#2286)

* chore: replace last_message_at with last_updated (#2282)

* chore: replace last_message_at with last_updated

* fix: keep the same order as before

* fix(expo): do not show reconnecting status while showing gallery

---------

Co-authored-by: Khushal Agarwal <[email protected]>
Co-authored-by: Arthur Geron <[email protected]>
Co-authored-by: Arthur Geron <[email protected]>
Co-authored-by: Vishal Narkhede <[email protected]>
@github-actions github-actions bot mentioned this pull request Nov 6, 2023
@stream-ci-bot
Copy link
Contributor

🎉 This PR is included in version 5.20.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.

Fix multiple mentions rendered incorrectly
3 participants