Skip to content

Add bot handle reporting to Telegram Transport#559

Merged
douglas-reid merged 1 commit intomainfrom
get-me
Sep 27, 2023
Merged

Add bot handle reporting to Telegram Transport#559
douglas-reid merged 1 commit intomainfrom
get-me

Conversation

@eob
Copy link
Copy Markdown
Contributor

@eob eob commented Sep 26, 2023

It looks like back in June, the reporting method which returned information about the bot itself was removed from the transport -- probably accidentally.

This is used to generate a link in the web UI that tells you which bot you've connected your agent to.

if ".steamship.run/" in api_base or ".apps.staging.steamship.com" in api_base:
# This is a special case for our testing pipeline -- it contains a mock Telegram server.
return api_base
return api_base[:-1]
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It turns out that even the testing one shouldn't have a / character..

)
return "OK"

@get("getMe", public=True)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Simulation of the telegram API endpoint.

@eob eob requested a review from douglas-reid September 27, 2023 00:01
@douglas-reid douglas-reid added this pull request to the merge queue Sep 27, 2023
Merged via the queue into main with commit cb70c9a Sep 27, 2023
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