ManiMozaffar / linkedIn-scraper

A playwright bot which is implemented to scrape linkedin and store advertisement data in a database and telegram channel
212 stars 22 forks source link

Rework For Telegram Section #5

Closed ZenMaxe closed 1 year ago

ZenMaxe commented 1 year ago

it's Better To Use Class-Based Pyrogram Client. i Added Filters & Commands Section, you can create new filters easily I prefer to use Wraps instead of creating new filters using Pyrogram because it's faster and better.

ManiMozaffar commented 1 year ago

Hey there, thanks for the PR :> There are some issues with your PR

  1. You did not followed snake_case naming style, which I fixed it on my commit, but I haven't pushed yet.
  2. For some weird reason as I have used wireshark it sent too many request to telegram for no reason causing 420 status code error but I could not see the content of request. Not sure if this is related to your PR or what, but then i guess i'll have to off bot for tonight and try again tomorrows
ZenMaxe commented 1 year ago

Hey there, thanks for the PR :> There are some issues with your PR

  1. You did not followed snake_case naming style, which I fixed it on my commit, but I haven't pushed yet.
  2. For some weird reason as I have used wireshark it sent too many request to telegram for no reason causing 420 status code error but I could not see the content of request. Not sure if this is related to your PR or what, but then i guess i'll have to off bot for tonight and try again tomorrows

Thanks For You're Tip. I Think its' Bug , just Restart the bot and set Workers to 1.