Skip to content

Recent #351

Open
Ruhonwtf wants to merge 6 commits intoCodeXBotz:mainfrom
Ruhonwtf:main
Open

Recent #351
Ruhonwtf wants to merge 6 commits intoCodeXBotz:mainfrom
Ruhonwtf:main

Conversation

@Ruhonwtf
Copy link

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates generated Telegram deep links and channel-post link parsing to use the telegram.me domain instead of t.me.

Changes:

  • Switched generated bot start links from https://t.me/... to https://telegram.me/... in link-generation and channel-post plugins.
  • Updated the DB channel post-link regex in get_message_id() to match https://telegram.me/....
  • Modified app.json formation settings (currently leaving the file structurally incomplete/invalid).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
plugins/link_generator.py Generates bot start links using telegram.me instead of t.me.
plugins/channel_post.py Generates bot start links for posted content using telegram.me.
helper_func.py Updates regex used to extract message IDs from Telegram post links to telegram.me.
app.json Alters worker formation configuration (but currently breaks JSON structure).
Comments suppressed due to low confidence (1)

app.json:87

  • app.json is now invalid JSON: the formation.worker object ends after "quantity": 1, with a trailing comma and missing closing braces. This will break platforms that parse app.json (e.g., Heroku/Railway deploy buttons). Restore the closing }/}/} and remove the trailing comma (and re-add size only if it’s still intended).
  "formation": {
    "worker": {
      "quantity": 1,


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines +73 to 74
pattern = "https://telegram.me/(?:c/)?(.*)/(\d+)"
matches = re.match(pattern,message.text)
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.

2 participants