Skip to content

fix: correct SetGroupDescription to properly handle deletes and updates#851

Open
xTudoS wants to merge 2 commits into
tulir:mainfrom
xTudoS:fix/set-group-description
Open

fix: correct SetGroupDescription to properly handle deletes and updates#851
xTudoS wants to merge 2 commits into
tulir:mainfrom
xTudoS:fix/set-group-description

Conversation

@xTudoS
Copy link
Copy Markdown

@xTudoS xTudoS commented Jun 18, 2025

This PR add missing attributes for description updates/deletions

Copy link
Copy Markdown
Contributor

@purpshell purpshell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Not sure why WhatsApp is doing this.. 🤔 Maybe it is to sequence these updates properly, and also probably just to track these changes (as always)

@xTudoS
Copy link
Copy Markdown
Author

xTudoS commented Aug 18, 2025

LGTM

Not sure why WhatsApp is doing this.. 🤔 Maybe it is to sequence these updates properly, and also probably just to track these changes (as always)

I think its a way to deprecate old devices and unofficial apps

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unrelated proto change

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unrelated proto change

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unrelated proto change

Comment thread group.go
}

content := waBinary.Node{
Tag: "description",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would look further to see if this is change that also affects topics etc. not just descriptions 👀

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you going to try? This PR is almost 1 year old

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.

2 participants