search

botman / driver-telegram

BotMan Telegram Driver
MIT License
87 stars 75 forks source link

Update Telegram Command #39

Closed RicardoRamirezR closed 6 years ago

RicardoRamirezR commented 6 years ago

If we are deploying it would be nice if we can pass a url in the command line to set the webhook. And there are some shortcuts added, -r to remove; -o for output, -url= for the optional url webhook.

mpociot commented 6 years ago

@RicardoRamirezR thank you for the PR. Can you take a look at the conflicts?

RicardoRamirezR commented 6 years ago

@mpociot done.

codecov[bot] commented 6 years ago

Codecov Report

Merging #39 into master will decrease coverage by 2.35%. The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master     #39      +/-   ##
===========================================
- Coverage     76.45%   74.1%   -2.36%     
- Complexity      118     121       +3     
===========================================
  Files            11      11              
  Lines           378     390      +12     
===========================================
  Hits            289     289              
- Misses           89     101      +12
Impacted Files Coverage Δ Complexity Δ
src/Console/Commands/TelegramRegisterCommand.php 0% <0%> (ø) 9 <5> (+3) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e42d0de...cda7f76. Read the comment docs.