search

Civcraft / Citadel

Do not open issues here; open them on the maintained fork @ DevotedMC
https://github.com/DevotedMC/Citadel
BSD 3-Clause "New" or "Revised" License
6 stars 23 forks source link

Provide group id in admin version of /cti #89

Closed erocs closed 9 years ago

erocs commented 9 years ago

Today the admin version of /cti shows the group name, but with NameLayer groups this could map to any number of group IDs. Please add the group number to the information.

erocs commented 9 years ago

That pull doesn't fix this.

rourke750 commented 9 years ago

https://github.com/Civcraft/Citadel/commit/d75674c365a2a0569c9856c0ca04f55592d938da https://github.com/Civcraft/Citadel/commit/0ca115e3894224c3a8357d60cd2db01bae23ad30#diff-35250cb092462f1ea9f47e16d8a7d5d5

erocs commented 9 years ago

Instead of displaying the group ID, this displays the internal string representation of the PlayerReinforcement object.

rourke750 commented 9 years ago

Already fixed in newest version. On Apr 25, 2015 6:28 PM, "erocs" notifications@github.com wrote:

Instead of displaying the group ID, this displays the internal string representation of the PlayerReinforcement object.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/Citadel/issues/89#issuecomment-96288696.