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

Create Telegram bot #355

Closed
wants to merge 1 commit into from
Closed

Create Telegram bot #355

wants to merge 1 commit into from

Conversation

Arjun00000
Copy link

Telegram extractor

Telegram extractor
@github-actions github-actions bot added the PR: unreviewed PR does not have any reviews. label Nov 7, 2024
Copy link
Owner

@EDM115 EDM115 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wth

@@ -0,0 +1,29 @@
import logging
from telegram.ext import Updater, CommandHandler, MessageHandler
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

which library is that ? not added to the requirements

import logging
from telegram.ext import Updater, CommandHandler, MessageHandler

TOKEN = 7901866660:AAFJ2Te56IAreu3nxnXKgRrQXNEKMF3CoSg #
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should NEVER give publicly any credentials, this violates point 4.5 of the Telegram Bot Platform Developer Terms of Service (https://telegram.org/tos/bot-developers#4-5-credentials)


logging.basicConfig(level=logging.INFO)

def save_restricted_content(update, context):
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's the purpose of this ?


def restrict_access(update, context):
# Restrict access to saved content
context.bot.send_message(chat_id=(link unavailable), text='Access restricted.')
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah yeah, the famous variable link unavailable

@EDM115
Copy link
Owner

EDM115 commented Nov 7, 2024

this has to be the wankiest PR I've ever seen :

  1. "Create Telegram bot" wdym, the repo already is a Telegram bot
  2. it adds no functionality
  3. it doesn't work with the current codebase
  4. it have credentials in plaintext (are you following Meta's or Replit's best practices lmao)
  5. the code just doesn't work

this PR won't be accepted
doesn't lgtm

@EDM115 EDM115 closed this Nov 7, 2024
@github-actions github-actions bot added PR: reviewed-changes-requested PR has reviewed and got Change request event. and removed PR: unreviewed PR does not have any reviews. labels Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: reviewed-changes-requested PR has reviewed and got Change request event.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants