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

Display MOTD by default on client #1157

Merged
merged 1 commit into from
May 10, 2017
Merged

Display MOTD by default on client #1157

merged 1 commit into from
May 10, 2017

Conversation

astorije
Copy link
Member

@astorije astorije commented May 9, 2017

Fixes #1019 (again).

This was changed in #1052 by @KlipperKyle but this change was lost during the switch to options module in #1066.

@astorije astorije added the Type: Bug Issues that report and PRs that solve any defects that cause unexpected behaviors. label May 9, 2017
@astorije astorije added this to the 2.3.0 milestone May 9, 2017
@astorije astorije requested a review from AlMcKinlay May 9, 2017 22:20
Copy link
Member

@AlMcKinlay AlMcKinlay left a comment

Choose a reason for hiding this comment

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

Ah sorry, these things are bound to happen when I wrote the code before you change this.

@AlMcKinlay
Copy link
Member

Minor fix for something which was approved previously. Merging.

@AlMcKinlay AlMcKinlay merged commit 07e4f4d into master May 10, 2017
@AlMcKinlay AlMcKinlay deleted the astorije/motd-default branch May 10, 2017 09:14
matburnham pushed a commit to matburnham/lounge that referenced this pull request Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Issues that report and PRs that solve any defects that cause unexpected behaviors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants