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

Group hierarchy improvements (by 3298) #7417

Merged
merged 3 commits into from Mar 27, 2019
Merged

Conversation

@LordAro
Copy link
Member

LordAro commented Mar 26, 2019

From https://www.tt-forums.net/viewtopic.php?f=33&t=83025

Group patches 1 & 2 have been previously implemented, so this is 3-5

Fix #5977
Fix #6053
Fix #6189

src/group_gui.cpp Outdated Show resolved Hide resolved
src/group_gui.cpp Outdated Show resolved Hide resolved
src/group_gui.cpp Outdated Show resolved Hide resolved
src/group_cmd.cpp Show resolved Hide resolved
@LordAro LordAro force-pushed the LordAro:3298-groups branch 2 times, most recently from 02e7446 to 2626b28 Mar 26, 2019
@LordAro LordAro requested a review from PeterN Mar 26, 2019
@LordAro

This comment has been minimized.

Copy link
Member Author

LordAro commented Mar 26, 2019

image
fixed.

@LordAro LordAro dismissed PeterN’s stale review Mar 26, 2019

fixed

@LordAro LordAro force-pushed the LordAro:3298-groups branch from 2626b28 to 4ac3ab5 Mar 26, 2019
@PeterN
PeterN approved these changes Mar 27, 2019
@PeterN PeterN merged commit 49f7332 into OpenTTD:master Mar 27, 2019
8 checks passed
8 checks passed
OpenTTD CI Build #20190326.20 succeeded
Details
OpenTTD CI (Linux commit-checker) Linux commit-checker succeeded
Details
OpenTTD CI (Linux linux-amd64-clang-3.8) Linux linux-amd64-clang-3.8 succeeded
Details
OpenTTD CI (Linux linux-amd64-gcc-6) Linux linux-amd64-gcc-6 succeeded
Details
OpenTTD CI (Linux linux-i386-gcc-6) Linux linux-i386-gcc-6 succeeded
Details
OpenTTD CI (MacOS) MacOS succeeded
Details
OpenTTD CI (Windows Win32) Windows Win32 succeeded
Details
OpenTTD CI (Windows Win64) Windows Win64 succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.