TShock icon indicating copy to clipboard operation
TShock copied to clipboard

Multiplayer packet names need to be consistent in documentation and in TSAPI/TShock

Open hakusaro opened this issue 6 years ago • 1 comments

Currently, the multiplayer packet docs refer to packets by names that don't match those in the PacketTypes enum and subsequent naming convention for packets. This is problematic as it makes referring to the packet documentation confusing because you have to go through PacketTypes to get the packet number, and then jump to the docs by number.

Preferably, the names in TShock should match those on the multiplayer packet documentation page.

hakusaro avatar Dec 20 '17 19:12 hakusaro

I also think their names should be rephrased and cleaned up eventually (with old names left in as obsolete entries of the enum). Example is https://github.com/Pryaxis/TerrariaAPI-Server/issues/159.

bartico6 avatar Jan 10 '18 13:01 bartico6