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

Contact driver #1

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Contact driver #1

wants to merge 22 commits into from

Conversation

hanc2006
Copy link
Owner

@hanc2006 hanc2006 commented Oct 7, 2018

Added missing TelegramContactDriver extension from TelegramDriver base class

@holtkamp
Copy link

I guess this can / should be merged so a PR can be sent to https://github.com/botman/driver-telegram ?

sh41 and others added 17 commits March 19, 2020 11:15
…re are no BC breaks.

When set to true this will perform various checks on the response from an http call and throw a TelegramConnectionException if there are any problems. The message content will contain debugging information.

Have added new tests for this.

Also fixed it_throws_exception_in_get_user test as it would not have correctly picked up situation where no exception was thrown.
…tyleci.io/standalone-php linter is not mentioned. Removed to fix

```
Config Issue
The provided config option 'linting' was not valid.
```
Optionally throw exception when http call fails.
* Fix bug in sending Contact and update tests to test both with and without a vcard.

* Cs Fixer

Co-authored-by: Steve Hall <[email protected]>
…ons to be sent with every request. Specifically this is to enable defaulting all messages to Markdown format by having the following config: (botman#70)

```
'telegram' => [
	'token' => 'YOUR-TELEGRAM-TOKEN-HERE',
	'default_additional_parameters' =>
	    [
	        'parse_mode' => 'Markdown',
	    ],
]
```

Co-authored-by: Steve Hall <[email protected]>
Co-authored-by: FeralHeart <[email protected]>
Use https instead of http.

Co-authored-by: antimech <[email protected]>
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.

5 participants