Skip to content

v9.7.2-beta.1

v9.7.2-beta.1: fix: segmented reactions on press (#3746)
@isekovanic isekovanic tagged this 29 Jul 20:26 
## 🎯 Goal

This PR addresses a behavioural change that unfortunately slipped
through with V9 that diverges from what we had in the previous versions.
Namely, for segmented reactions we applied reaction toggling instead of
targeted opening of the reaction info bottom sheet.

This diverged from the other SDKs and so here we align this divergence.

Additionally, we pass `reactionType` to `additionalInfo` and make sure
it's propagated everywhere so that we can easily replicate the behaviour
if needed, like so:

```
<Channel
  onPressMessage={({ emitter, additionalInfo, actionHandlers, defaultHandler }) => {
    if (emitter === 'reactionList' && additionalInfo?.reactionType) {
      actionHandlers?.toggleReaction(additionalInfo.reactionType as string);
      return;
    }
    defaultHandler?.();
  }}
/>
```

## 🛠 Implementation details

<!-- Provide a description of the implementation -->

## 🎨 UI Changes

<!-- Add relevant screenshots -->

<details>
<summary>iOS</summary>


<table>
    <thead>
        <tr>
            <td>Before</td>
            <td>After</td>
        </tr>
    </thead>
    <tbody>
        <tr>
            <td>
                <!--<img src="" /> -->
            </td>
            <td>
                <!--<img src="" /> -->
            </td>
        </tr>
    </tbody>
</table>
</details>


<details>
<summary>Android</summary>

<table>
    <thead>
        <tr>
            <td>Before</td>
            <td>After</td>
        </tr>
    </thead>
    <tbody>
        <tr>
            <td>
                <!--<img src="" /> -->
            </td>
            <td>
                <!--<img src="" /> -->
            </td>
        </tr>
    </tbody>
</table>
</details>

## 🧪 Testing

<!-- Explain how this change can be tested (or why it can't be tested)
-->

## ☑️ Checklist

- [ ] I have signed the [Stream
CLA](https://docs.google.com/forms/d/e/1FAIpQLScFKsKkAJI7mhCr7K9rEIOpqIDThrWxuvxnwUq2XkHyG154vQ/viewform)
(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
Assets 2
Loading