Added Phonenumber to SyncResult Event #873
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Also fixes venomous0x#492
Repaired onProfilePictureChanged event, added onProfilePictureDeleted
Typos fixed as well as incorrectly commented function name
…urrent behavior and it fix the problem if WhatsAPI is being used from other project/path
added full path on the fopen(countries.csv) line,
onGetRequestLastSeen return an array with 4 elements, not with 3!
Update EVENTS.md
…urrent behavior and it fix the problem if WhatsAPI is being used from other project/path
onGetRequestLastSeen return an array with 4 elements, not with 3!
added onGetGroupMessage
Update EVENTS.md
Some minimun details
Now you can send media without downloading the file, just knowing the size and the base64 hash. TODO: some minor fixes in func.php probably tomorrow
Now broadcast media knowing only size and hash
Send media files knowing only the size and hash - Enhancement venomous0x#826
new ping request from server is like this ```<iq from="s.whatsapp.net" id="1405327577-ping" type="get" xmlns="urn:xmpp:ping"></iq>```
ping response from server changed Fix from venomous0x#700
Update event to let the user get filehash
Added Simple CLI and fixes to exampleFunctional
If number add to group
Update whatsprot.class.php
profile picture update notification node now without message node
update profile picture update notification
|
May I know which one should be enhanced.. |
| $callbackEvent = function(WhatsAppEventListener $listener) use ($result) { | ||
| $listener->onGetSyncResult($result); | ||
| $callbackEvent = function(WhatsAppEventListener $listener) use ($phone,$result) { | ||
| $listener->onGetSyncResult($phone,$result); |
There was a problem hiding this comment.
This changes the Public API of the onGetSyncResult event. It would be better to add the phone number to the SyncResult class instead.
hubail
force-pushed
the
master
branch
2 times, most recently
from
d56e5ea
to
e6cf5a5
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added Phonenumber to SyncResult Event.