Merge lp://staging/~widelands-dev/widelands/bug-1827275-join_message_irc-lobby into lp://staging/widelands

Proposed by Toni Förster
Status: Merged
Merged at revision: 9091
Proposed branch: lp://staging/~widelands-dev/widelands/bug-1827275-join_message_irc-lobby
Merge into: lp://staging/widelands
Diff against target: 12 lines (+1/-1)
1 file modified
src/network/internet_gaming.cc (+1/-1)
To merge this branch: bzr merge lp://staging/~widelands-dev/widelands/bug-1827275-join_message_irc-lobby
Reviewer Review Type Date Requested Status
GunChleoc Approve
Review via email: mp+366790@code.staging.launchpad.net

Commit message

check for client.type to not confuse IRC and lobby names

To post a comment you must log in.
Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 4850. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/527041876.
Appveyor build 4631. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1827275_join_message_irc_lobby-4631.

Revision history for this message
GunChleoc (gunchleoc) wrote :

LGTM :)

@bunnybot merge

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: