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

Feature/platform docker #154

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Thlb
Copy link
Contributor

@Thlb Thlb commented Apr 24, 2020

[ ] I'm submitting a template app

  • I have added a .ini file to DashMachine with the same name used for [App Name] in the .ini file (e.g. 'App Name.ini')
  • My .ini has the exact format as the others in the folder, only changing the name, icon location, and description.
  • I used the official description and name for the app, found on their repository/website.
  • I chose an icon (.png file) sized over 64px, with a transparent background, square aspect ratio, preferably the icon only version (not icon w/ text) of the app's icon.
  • I understand that the icon will be resized to 64px x 64px and have verified that it looks good at that size.
  • I have added the icon to static/images/apps with the correct name matching what's in the .ini.
  • I tested it to make sure it looks good in the interface

[ ] I'm submitting a platform

  • I looked at the other platform examples and followed a very similar methodology.
  • I added a docstring at the top of my platform file that is formatted exactly like rest.py
  • I added sample code to any template app that this platform works for, look at deluge.ini in /template_apps/ for an example
  • I have thoroughly tested this platform and it works
  • I am committed to updating this platform if something breaks it in the future

@Thlb
Copy link
Contributor Author

Thlb commented Apr 24, 2020

Add docker platform :

  • Can be used to monitor containers on a Docker endpoint.
  • Connection to docker API can be made with TLS using certificates (cert / key / ca)
  • Two display mode : on App card (e.g. Portainer), or custom card

App card preview :
app-card
Custom card preview :
custom-card
Custom card (data) preview :
custom-card-data

@rmountjoy92
Copy link
Owner

THIS IS AWESOME! sorry for the late reply. Just wanted to let you know that merging pull request in v0.6 is high on my list of priorities. It's not an easy as usual, because there are a lot of major back end changes in v0.6, so I will need to edit all PR's to be compatible with the new version. I will close this PR once this is added to 0.6 :)

@rmountjoy92
Copy link
Owner

@Thlb same here. (super stoked about this one. I'm gonna set up mine with my docker install today and prolly make a post about it on the Reddit)

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