Skip to content

Contact driver#1

Open
hanc2006 wants to merge 22 commits intomasterfrom
contact-driver
Open

Contact driver#1
hanc2006 wants to merge 22 commits intomasterfrom
contact-driver

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 <steve.hall+gitlab@rg456.co.uk>
…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 <steve.hall+gitlab@rg456.co.uk>
Co-authored-by: FeralHeart <feralheart@feralsystems.com>
Co-authored-by: antimech <antimech@users.noreply.github.com>
Co-authored-by: antimech <antimech@users.noreply.github.com>
Use https instead of http.

Co-authored-by: antimech <antimech@users.noreply.github.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.

5 participants