-
-
Notifications
You must be signed in to change notification settings - Fork 88
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
Deprecate webxdc sendUpdate's desc parameter #6245
Labels
Comments
see also webxdc/website#92 which removed "descr"
for the next spec release.
|
@Septias i already did a little PR, i overlooked you were assigned |
r10s
added a commit
that referenced
this issue
Nov 24, 2024
this PR removes most usages of the `descr` parameter. - to avoid noise in different branches etc. (as annoying on similar, at a first glance simple changes), i left the external API stable - also, the effort to do a database migration seems to be over the top, so the column is left and set to empty strings on future updates - maybe we can recycle the column at some point ;) closes #6245
This was referenced Nov 24, 2024
adbenitez
added a commit
to deltachat/deltachat-android
that referenced
this issue
Nov 24, 2024
the 'description' parameter is about to be deprecated, see deltachat/deltachat-core-rust#6245
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
as commented in #6230 (comment)
the
desc
field is unnecessary, was never needed/used by other implementators that raised questions about it, ex. in XMPP, it was a DeltaChat-specific thing mainly, to show something nice to a classic email user, but that doesn't make much sense anyways as status updates can be a lot and are internal data not supposed to be analyzed or described manuallythis is a backwards compatible change since in javascript you can call functions with arbitrary amount of extra arguments and they will just be ignored/unused
The text was updated successfully, but these errors were encountered: