Skip to content

fix: [REACT-353] unify pinned_at & archived_at nullish values#1516

Merged
arnautov-anton merged 1 commit intomasterfrom
fix/pinned-archived-null-to-undefined
Apr 17, 2025
Merged

fix: [REACT-353] unify pinned_at & archived_at nullish values#1516
arnautov-anton merged 1 commit intomasterfrom
fix/pinned-archived-null-to-undefined

Conversation

@arnautov-anton
Copy link
Contributor

Description of the changes, What, Why and How?

Event handler for member.updated receives a member object which contains archived_at and pinned_at with null values if unarchive/unpin actions are called - to make matters for our integrators easier, the only nullish value for these would be undefined. I'll keep the types unchanged (#1515) as the ChannelMemberResponse used in the Event type is still correct.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 14, 2025

Size Change: +163 B (+0.03%)

Total Size: 535 kB

Filename Size Change
dist/browser.es.js 117 kB +38 B (+0.03%)
dist/browser.full-bundle.min.js 63.9 kB +5 B (+0.01%)
dist/browser.js 118 kB +38 B (+0.03%)
dist/index.es.js 117 kB +41 B (+0.03%)
dist/index.js 118 kB +41 B (+0.03%)

compressed-size-action

@arnautov-anton arnautov-anton force-pushed the fix/pinned-archived-null-to-undefined branch 6 times, most recently from 9ccb0e2 to 72b5bfb Compare April 17, 2025 10:19
@arnautov-anton arnautov-anton force-pushed the fix/pinned-archived-null-to-undefined branch from 72b5bfb to 3e05e4f Compare April 17, 2025 10:19
@szuperaz
Copy link
Contributor

@arnautov-anton won't we still receive null in queryChannels response?

@arnautov-anton
Copy link
Contributor Author

arnautov-anton commented Apr 17, 2025

@arnautov-anton won't we still receive null in queryChannels response?

No, @szuperaz, it seems that this sort of data is only delivered through events.

queryChannels after pinning:
image
after unpinning (and re-querying):
image

@arnautov-anton arnautov-anton merged commit a840226 into master Apr 17, 2025
5 checks passed
@arnautov-anton arnautov-anton deleted the fix/pinned-archived-null-to-undefined branch April 17, 2025 12:55
@github-actions github-actions bot mentioned this pull request Apr 17, 2025
arnautov-anton pushed a commit that referenced this pull request Apr 17, 2025
# 🚀 v8.60.0
  Make sure to use squash & merge when merging!
Once this is merged, another job will kick off automatically and publish
the package.
  # 📝 Changelog
##
[8.60.0](v8.59.0...v8.60.0)
(2025-04-17)


### Features

* [CHA-794] Add sort and filter param to queryThreads
([#1511](#1511))
([ea7fe99](ea7fe99))
* [CHA-855] - Refactoring partial update member
([#1517](#1517))
([e4f7e68](e4f7e68))


### Bug Fixes

* [REACT-350] make archived_at & pinned_at nullable
([#1515](#1515))
([318825a](318825a))
* [REACT-353] unify pinned_at & archived_at nullish values
([#1516](#1516))
([a840226](a840226)),
closes [#1515](#1515)

Co-authored-by: github-actions <[email protected]>
github-actions bot pushed a commit that referenced this pull request Apr 28, 2025
## [9.0.0-rc.11](v9.0.0-rc.10...v9.0.0-rc.11) (2025-04-28)

### ⚠ BREAKING CHANGES

* Replacement of FormatMessageResponse with LocalMessage
type

### Bug Fixes

* [REACT-344] remove Agora & 100ms integrations ([#1519](#1519)) ([16cd81a](16cd81a))
* [REACT-350] make archived_at & pinned_at nullable ([#1515](#1515)) ([318825a](318825a))
* [REACT-353] unify pinned_at & archived_at nullish values ([#1516](#1516)) ([a840226](a840226)), closes [#1515](#1515)

### Features

* [CHA-794] Add sort and filter param to queryThreads ([#1511](#1511)) ([ea7fe99](ea7fe99))
* [CHA-855] - Refactoring partial update member ([#1517](#1517)) ([e4f7e68](e4f7e68))
* message composer ([#1495](#1495)) ([0c07524](0c07524)), closes [stream-chat-react#2669](GetStream/stream-chat-react#2669)
@stream-ci-bot
Copy link

🎉 This PR is included in version 9.0.0-rc.11 🎉

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants