driver-telegram icon indicating copy to clipboard operation
driver-telegram copied to clipboard

add getUserProfilePhotos method

Open feralheart opened this issue 7 years ago • 7 comments

I made these changes for the Issue #33 Documentation:

feralheart avatar Mar 24 '18 13:03 feralheart

Codecov Report

Merging #35 into master will decrease coverage by 3.2%. The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #35      +/-   ##
============================================
- Coverage     77.68%   74.48%   -3.21%     
- Complexity      116      121       +5     
============================================
  Files            11       11              
  Lines           372      388      +16     
============================================
  Hits            289      289              
- Misses           83       99      +16
Impacted Files Coverage Δ Complexity Δ
src/TelegramDriver.php 84.88% <0%> (-8.71%) 50 <5> (+5)

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 45ed1e8...759da85. Read the comment docs.

codecov[bot] avatar Mar 24 '18 13:03 codecov[bot]

@feralheart Thank you for the PR. Hm I wonder if we should add this to the Telegram User object.

mpociot avatar Apr 05 '18 09:04 mpociot

@mpociot You're welcome. I thinked about it, too, but I didn't want to break the getUser method.

feralheart avatar Apr 06 '18 21:04 feralheart

I don't think that we break it. The Telegram driver already has a custom user object that gets returned: https://github.com/botman/driver-telegram/blob/master/src/Extensions/User.php So we could add a setter / getter for the image and pass the image to the object when it gets created.

mpociot avatar Apr 09 '18 06:04 mpociot

It's a good idea. I will do it :)

feralheart avatar Apr 14 '18 18:04 feralheart

@mpociot so? Merge maybe?

antimech avatar Dec 12 '18 16:12 antimech

@feralheart any upd?

antimech avatar Feb 27 '19 13:02 antimech

Move this to the User object, replace the env() call with $this->config() and I'll approve it.

filippotoso avatar Jun 24 '24 04:06 filippotoso