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

Added Plugin MQTT #498

Merged
merged 8 commits into from
Jan 7, 2021
Merged

Added Plugin MQTT #498

merged 8 commits into from
Jan 7, 2021

Conversation

mrduckspace
Copy link
Contributor

Plugin für die Übermittlung von POSAC an einem MQTT-Broker

@Schrolli91
Copy link
Owner

Schrolli91 commented Dec 18, 2020

Vielen Dank für deinen Beitrag - ich versuch am Wochenende ein Review zu machen :-)

Kleine Info fürs nächste mal:
Bitte die Änderungen auf dem develop Branch durchführen und den Pull Request dann auch gegen den develop stellen. Habe das eben angepasst - Nur zur Info

@Schrolli91 Schrolli91 self-requested a review December 18, 2020 18:14
@Schrolli91 Schrolli91 changed the base branch from master to develop December 18, 2020 18:14
@mrduckspace
Copy link
Contributor Author

Hey,
Immer gerne, finde deine Arbeit sehr gut. Falls du Mal Hilfe brauchst, melde dich gerne

Ja hab ich letzte Mal auch gemacht, hab es aber heute leider vergessen :D

Copy link
Owner

@Schrolli91 Schrolli91 left a comment

Choose a reason for hiding this comment

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

LGTM

Hättest du Lust, FMS und ZVEI noch zu implementieren?
Falls nicht, würde ich es im aktuellen Stand mergen

@mrduckspace
Copy link
Contributor Author

Kann ich wohl machen, habe aber ehrlich gesagt keine Ahnung von FMS und ZVEI. Würde dann einfach die Wildcards (die in der Config unten stehen) als JSON per MQTT übergeben oder? @Schrolli91

wird aber wahrscheinlich erst im Laufe der Woche was

@Schrolli91
Copy link
Owner

Ja genau, daran dachte ich. Einfach die Felder - wie mein POCSAG auch - in ein JSON packen und versenden.
Super Danke :-)

@mrduckspace
Copy link
Contributor Author

hab es doch jetzt schon geschafft :-) hoffe es passt so :P

config/config.template.ini Outdated Show resolved Hide resolved
@mrduckspace
Copy link
Contributor Author

hab ich vorhin beim Ändern vergessen, ist geändert.

Copy link
Owner

@Schrolli91 Schrolli91 left a comment

Choose a reason for hiding this comment

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

LGTM

CHANGELOG.md Outdated Show resolved Hide resolved
@Schrolli91 Schrolli91 merged commit b3d3e1c into Schrolli91:develop Jan 7, 2021
@mrduckspace
Copy link
Contributor Author

hey danke @Schrolli91 ;) der Changelog ist bei mir untergegangen

@Schrolli91
Copy link
Owner

Kein Problem ;-) Hatte heut mal eben die zeit gefunden ;-)

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