Skip to content
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

added botInvokeMessage to normalizeMessageContent #1237

Closed
wants to merge 1 commit into from

Conversation

Skidy89
Copy link

@Skidy89 Skidy89 commented Feb 3, 2025

its a futureproof message so yeah.
also this are futureproof as well
associatedChildMessage
groupStatusMentionMessage
pollCreationMessageV4
pollCreationMessageV5
statusAddYours
groupStatusMessage

@whiskeysockets-bot
Copy link
Contributor

Thanks for your contribution.

The next step is to wait for review and approval to merge it to main repository

The community can help reacting with a thumb up (:thumbsup:) for approval and rocket (:rocket:) for who has tested it.

To test this PR you can run the following command below:

# NPM
npm install @whiskeysockets/baileys@Skidy89/Baileys-fork#master
# YARN v2
yarn add @whiskeysockets/baileys@Skidy89/Baileys-fork#master

@azudindaem
Copy link
Contributor

I think missing something here.. need to update WAProto to support groupStatusMentionMessage

@Skidy89
Copy link
Author

Skidy89 commented Feb 4, 2025

I think missing something here.. need to update WAProto to support groupStatusMentionMessage

Can i upload the protos, but purp say he would refactor it

@azudindaem
Copy link
Contributor

@purpshell .. we should add proto v3 soon

@purpshell
Copy link
Collaborator

protos are due the update from the protococktail

@purpshell
Copy link
Collaborator

I'd prefer if message types like these were only added in the context of fuller implementations, for example this should be part of #1252)

@purpshell purpshell closed this Feb 24, 2025
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.

4 participants