search

Slanfan / MTG-Tolaria

8 stars 1 forks source link

Deck name requirement #491

Closed Slanfan closed 4 months ago

Slanfan commented 9 months ago

I don't know if it is intentional, but it's possible to have the deck name for a deck be ' ' as the check for the 5 character length requirement doesn't trim whitespace before getting the length of the name.

If it is intentional, or not really a concern, then... hi!

Reporter: Elijah Hartvigsen
ReporterUid: 0umcVLht9vUDggkO95LuEx3wh9P2
RpoerterPlayerId: 8b741907-5151-4d9e-92c0-a0ab61abc98c
Slanfan commented 4 months ago

https://github.com/Slanfan/Tolaria/pull/270/commits/65c2475bbf666ef5638e04598f63ba9e2368d1fb