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

Update Telegram.py #382

Merged
merged 7 commits into from
Nov 8, 2018
Merged

Update Telegram.py #382

merged 7 commits into from
Nov 8, 2018

Conversation

PeterLaemmle
Copy link
Contributor

-Add routing per overlay in overview map
-Switch to SSL communication

-Add routing per overlay in overview map
-Switch to SSL communication
@flothi
Copy link
Collaborator

flothi commented Oct 28, 2018

Ich würde die Definition des Origins in das config-File setzen, ggf sollten wir auch den generischen String für die Location in die config setzen

Allow definition of routing origin in configuration file
Allow definition of routing origin in configuration file
Pleasing the Codacy BOT ;-)
@Schrolli91
Copy link
Owner

Changelog bitte nicht vergessen

@Schrolli91
Copy link
Owner

@PeterLaemmle Kann in der aktuellen Iteration als "Fertig" angesehen und gemerged werden?

@PeterLaemmle
Copy link
Contributor Author

Ja.
Funktional war schon der erste Commit fertig. Bei den nachfolgenden kamen nur noch Änderungswünsche aus dieser Diskussion und von Codacy rein.
Wenn die Änderungen dazu noch von jemandem hier gereviewed und getestet wurden umso besser.

@Schrolli91
Copy link
Owner

Super - Vielen Dank für deine Arbeit

@Schrolli91 Schrolli91 merged commit 11e948e into Schrolli91:develop Nov 8, 2018
@ghost ghost removed the Review needed label Nov 8, 2018
@PeterLaemmle PeterLaemmle deleted the develop branch November 8, 2018 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants