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

refactor telegram #11

Merged
merged 3 commits into from
Jul 8, 2019
Merged

refactor telegram #11

merged 3 commits into from
Jul 8, 2019

Conversation

tanneberger
Copy link

No description provided.

@tanneberger tanneberger force-pushed the experimental branch 2 times, most recently from 94b2ef9 to 5f2a8e6 Compare July 7, 2019 19:08
@@ -3,7 +3,7 @@
import shutil
from datetime import datetime
from sqlalchemy import func
from typing import Iterator, List, Union
from typing import Iterator, List, Union, Tuple
Copy link
Contributor

Choose a reason for hiding this comment

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

T kommt vor U 😉

Copy link
Author

Choose a reason for hiding this comment

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

jukt keinen

app/func/essentials.py Show resolved Hide resolved

@run_async
def upvote(bot: Bot, update: Update) -> None:
query = update.callback_query
Copy link
Contributor

Choose a reason for hiding this comment

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

Fehlt ein Typ


def init_telegram():

updater = Updater(config["telegram_token"])
Copy link
Contributor

Choose a reason for hiding this comment

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

Fehlt noch ein Typ

uuid>=1.30
python-telegram-bot>=11.1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing new line

@@ -53,7 +53,7 @@ services:
destination: /app/app/data/
read_on_start: 'True'
# Tokens required
telegram_token:
Copy link
Contributor

Choose a reason for hiding this comment

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

Geht gar nicht

@SuperSandro2000
Copy link
Contributor

Kann ich auch morgen ändern.

@SuperSandro2000 SuperSandro2000 merged commit 2c3ad7d into master Jul 8, 2019
@SuperSandro2000 SuperSandro2000 deleted the experimental branch July 8, 2019 10:47
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