Skip to content
This repository has been archived by the owner on Oct 30, 2019. It is now read-only.

Add MOTD to the Hassbian image #201

Merged
merged 10 commits into from
Oct 23, 2018
Merged

Add MOTD to the Hassbian image #201

merged 10 commits into from
Oct 23, 2018

Conversation

Landrash
Copy link
Collaborator

Description:

Adds a MOTD on logon either over ssh or local based on the work of @yboetz.
Includes an update to dependencies also.

Name printed is derived from hostname of the machine. If changed this will also change.
Services are so far a static list but could be reworked to be added by hassbian-config but should not be governed by this PR since that would require changes to the tool in question.

screenshot_20181021_234254

Checklist (Required):

Copy link
Member

@ludeeus ludeeus left a comment

Choose a reason for hiding this comment

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

I have added some suggestions and a comment on this :)
It generally looks good 👍

package/etc/update-motd.d/30-services Outdated Show resolved Hide resolved
package/etc/update-motd.d/30-services Outdated Show resolved Hide resolved
package/etc/update-motd.d/30-services Outdated Show resolved Hide resolved
package/etc/update-motd.d/30-services Outdated Show resolved Hide resolved
package/etc/update-motd.d/30-services Outdated Show resolved Hide resolved
@@ -1,8 +1,8 @@
Package: hassbian-scripts
Version: 0.9.1
Version: 0.9.3
Copy link
Member

Choose a reason for hiding this comment

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

Was this change intentional?

package/etc/update-motd.d/30-services Outdated Show resolved Hide resolved
Copy link
Member

@ludeeus ludeeus left a comment

Choose a reason for hiding this comment

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

🐬

@ludeeus ludeeus merged commit 2ee5a9f into dev Oct 23, 2018
@ludeeus ludeeus deleted the motd branch October 23, 2018 17:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants