Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add '_is_valid_uuid()' to '_resolve_receiver()' for users without number #65

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

Kariton
Copy link
Contributor

@Kariton Kariton commented Jun 2, 2024

fixes #48

@Era-Dorta
Copy link
Collaborator

In case it's useful for anybody, for sending messages to UUIDS, the image in this issue is needed on top of this bbernhard/signal-cli-rest-api#550 (comment)

@Kariton
Copy link
Contributor Author

Kariton commented Jul 13, 2024

interesting.
i am able to sent to UUIDs with bbernhard/signal-cli-rest-api:0.84

thats why i submitted this PR in the first place.

@Era-Dorta
Copy link
Collaborator

Maybe it had to do with using the bot functions reply vs send. I didn't investigate further but the fix in that issue got released for version bbernhard/signal-cli-rest-api:0.85. So, I guess it doesn't really matter anyways.

@filipre filipre merged commit 4d45f2e into filipre:master Aug 5, 2024
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New accounts dont have phone numbers, _is_phone_number receives null and fails
3 participants