Skip to content

api!: jsonrpc: device message api now requires Option<MessageData> instead of String for the message#5211

Merged
Simon-Laux merged 2 commits intomainfrom
simon/jsonrpc-update-device-message-api
Jan 25, 2024
Merged

api!: jsonrpc: device message api now requires Option<MessageData> instead of String for the message#5211
Simon-Laux merged 2 commits intomainfrom
simon/jsonrpc-update-device-message-api

Conversation

@Simon-Laux
Copy link
Copy Markdown
Member

@Simon-Laux Simon-Laux commented Jan 24, 2024

api: jsonrpc: device message api now setting empty device messages to block adding a message for a specific label

required for deltachat/deltachat-desktop#3639

…instead of `String` for the message

api: jsonrpc: device message api now setting empty device messages to block adding a message for a specific label
Co-authored-by: iequidoo <117991069+iequidoo@users.noreply.github.com>
@Simon-Laux Simon-Laux merged commit 19dce9d into main Jan 25, 2024
@Simon-Laux Simon-Laux deleted the simon/jsonrpc-update-device-message-api branch January 25, 2024 10:26
adzialocha pushed a commit to deltachat/deltachat-desktop that referenced this pull request Feb 1, 2024
adzialocha pushed a commit to deltachat/deltachat-desktop that referenced this pull request Feb 1, 2024
Simon-Laux added a commit to deltachat/deltachat-desktop that referenced this pull request Feb 3, 2024
* do not add changelog device message on fresh accounts

closes #3637

depends on chatmail/core#5211

* changelog entry

* apply code review suggestions

* Remove dead code

---------

Co-authored-by: adz <x12@adz.garden>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants