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

acceptGroupV4Invite #677

Merged
merged 11 commits into from
Jun 5, 2021
Merged

acceptGroupV4Invite #677

merged 11 commits into from
Jun 5, 2021

Conversation

Darker935
Copy link
Contributor

No description provided.

Copy link
Owner

@pedroslopez pedroslopez left a comment

Choose a reason for hiding this comment

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

Cool!

This makes me think of a couple questions:

  • Do the old invites still work?
  • How do these differ from the old invites?
  • Could something be done to make the existing acceptInvite function work with the new format?

Also, it seems like typings are missing for the new acceptGroupV4Invite function.

@Darker935
Copy link
Contributor Author

Cool!

This makes me think of a couple questions:

* Do the old invites still work?

* How do these differ from the old invites?

* Could something be done to make the existing `acceptInvite` function work with the new format?

Also, it seems like typings are missing for the new acceptGroupV4Invite function.

1° - Yes, old invites are links, that anyone can join using them
2° - The InviteV4 is a private invite that only the user can use (it is received when the user does not allow to be added to groups by not contacts)
3° - The new method requires additional parameters, like expiration time, invite code, the user who received and who sent, that are in the message data

@Darker935 Darker935 requested a review from pedroslopez June 4, 2021 17:00
@pedroslopez
Copy link
Owner

Thanks! Tested this and it worked great. Just made a few changes to the typings.

@pedroslopez pedroslopez merged commit cdc00e9 into pedroslopez:master Jun 5, 2021
harish2704 added a commit to harish2704/whatsapp-web.js that referenced this pull request Jun 9, 2021
# By Pedro S. Lopez (3) and others
# Via GitHub
* 'master' of github.com:harish2704/whatsapp-web.js:
  Update lint.yml
  feat: acceptGroupV4Invite (pedroslopez#677)
  chore: update eslint and ecmaVersion (pedroslopez#680)
  chore: update lint action (pedroslopez#679)
  feat(dev): add argument bypassCSP to use in pupperter (pedroslopez#635)
  add searchMessages method (pedroslopez#586)
  feat: Add from which device the message was sent (pedroslopez#648)
  feat: disable / enable WA Web features (pedroslopez#543)
  feat: Get Orders and Products (pedroslopez#612)
  add extra option to MessageSendOptions (pedroslopez#600)
  feat: openChatWindow can open new chats (pedroslopez#592)
  Add sendVideoAsGif option (pedroslopez#578)
  Add support for sticker Name and author (pedroslopez#527)

# Conflicts:
#	package.json
Belfio pushed a commit to Belfio/whatsapp-web.js that referenced this pull request Jan 9, 2024
* Added option client.acceptGroupV4Invite()

* Update Client.js

* Update index.d.ts

Added a description to the inviteV4 object

Renamed "invite" by "inviteV4"

* Update Client.js

* Update Message.js

renamed "invite" to "inviteV4"

* Update Client.js

* Added message.acceptGroupV4Invite method

* Update index.d.ts

acceptGroupV4Invite

* better typings, refactor some names

Co-authored-by: Pedro Lopez <pedroslopez@me.com>
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