Skip to content

Conversation

Simon-Laux
Copy link
Contributor

@Simon-Laux Simon-Laux commented Oct 8, 2025

trying the breaking change.
Actually it will be not as breaking if you used the constants, because this pr will also change the constants.

closes #7029

Note that I had to change the constants from enum to namespace, this has the side effect, that you can no longer also use the constants as types, you need to instead prefix them with typeof now.

enum/union. Affected places: `FullChat.chat_type`,
`BasicChat.chat_type`,
`ChatListItemFetchResult::ChatListItem.chat_type`, `Event::
SecurejoinInviterProgress.chat_type` and `MessageSearchResult.chat_type`
@Simon-Laux Simon-Laux self-assigned this Oct 8, 2025
@Simon-Laux Simon-Laux marked this pull request as ready for review October 11, 2025 15:54
@Simon-Laux Simon-Laux removed their assignment Oct 16, 2025
@iequidoo iequidoo changed the title api!: jsonrpc: chat_type now contains a variant of a string enum/union. Affected places: FullChat.chat_type, BasicChat.chat_type, ChatListItemFetchResult::ChatListItem.chat_type, Event:: SecurejoinInviterProgress.chat_type and MessageSearchResult.chat_type api!(jsonrpc): chat_type now contains a variant of a string enum/union. Affected places: FullChat.chat_type, BasicChat.chat_type, ChatListItemFetchResult::ChatListItem.chat_type, Event:: SecurejoinInviterProgress.chat_type and MessageSearchResult.chat_type Oct 20, 2025
@Hocuri
Copy link
Collaborator

Hocuri commented Oct 21, 2025

Note that merging this will have to wait until we UIs can safely update core again (i.e. at least until #7326 is merged, better until we have another core release from main). Otherwise, building Desktop main with core main will fail for an extended period of time.

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.

Stronger TypeScript types of number (C-like) enums (e.g. chat_type)

3 participants