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 app MQTT page #83

Merged
merged 7 commits into from
May 6, 2022
Merged

Update app MQTT page #83

merged 7 commits into from
May 6, 2022

Conversation

Jaxom99
Copy link
Contributor

@Jaxom99 Jaxom99 commented Apr 28, 2022

I found 1 error in the code (escaping the $SYS topic) and then I proposed some style changes for better understanding.

Jaxom99 added 4 commits April 28, 2022 16:35
1 error in the code (escaping the $SYS topic) and some style changes for better understanding.
Escaping and .md interpretation allow for some errors in code afterwards. This is a proposal for improvement :)
same errors
@Jaxom99
Copy link
Contributor Author

Jaxom99 commented Apr 28, 2022

I change several files because I don't know how content is generated, and which files depend on which... Anyway feel free to correct me!

Jaxom99 added 3 commits April 28, 2022 17:11
remove just one /
Same error
same errors
@Jaxom99
Copy link
Contributor Author

Jaxom99 commented Apr 28, 2022

And also some typos in a topic settings, resulting in timeout errors (since no message is received on the faulty topic)

@oscar120584 oscar120584 merged commit 178d91c into zabbix:main May 6, 2022
@Jaxom99
Copy link
Contributor Author

Jaxom99 commented May 6, 2022

👍

@Jaxom99 Jaxom99 deleted the patch-1 branch May 6, 2022 10:00
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