Skip to content

Feat: Rephrase for a more relaxed notification style, see #42#43

Merged
rimusz merged 1 commit intoTheNewNormal:devfrom
franz-josef-kaiser:issue-42-notification-style
Nov 23, 2016
Merged

Feat: Rephrase for a more relaxed notification style, see #42#43
rimusz merged 1 commit intoTheNewNormal:devfrom
franz-josef-kaiser:issue-42-notification-style

Conversation

@franz-josef-kaiser
Copy link
Copy Markdown

  • Removes the !!! shouting,
  • Removes unnecessary space usage,
  • Tries to to use Corectl instead of the corectl variant found here and there
  • Some rephrasing
  • Be nicer: Say please
  • Do not panic – we are dealing with developers (who do not read anyway) – when a VM goes down, then it goes dow
  • A little bit of grammar and such things here and there
  • Consolidates; Server is Off and Server is running into Server status: Off/Running
  • Replaces Active VMs: 0 with No Active VMs
  • … minor things

Hope you like it.

@franz-josef-kaiser franz-josef-kaiser changed the base branch from master to dev November 21, 2016 22:18
@franz-josef-kaiser
Copy link
Copy Markdown
Author

Puh … realized the wrong target branch (master) when I hit the "send" button. The target branch now is dev for a test run.

@rimusz
Copy link
Copy Markdown
Member

rimusz commented Nov 22, 2016

awesome, thanks. will test it out next when i'm back from holiday:)

@rimusz rimusz merged commit 6fb30ae into TheNewNormal:dev Nov 23, 2016
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