Make a command class in deck creator. #22

Closed
opened 2023-04-11 08:54:53 +00:00 by LunarTides · 3 comments
LunarTides commented 2023-04-11 08:54:53 +00:00 (Migrated from github.com)

The deck creator has a lot of commands, so it should have a better way of managing them than a lot of if statements. Maybe make a command class and have each command be an object from this class. You can figure out the details.

The deck creator has a lot of commands, so it should have a better way of managing them than a lot of if statements. Maybe make a command class and have each command be an object from this class. You can figure out the details.
LunarTides commented 2023-06-23 19:27:22 +00:00 (Migrated from github.com)

Will get fixed in #192

Will get fixed in #192
LunarTides commented 2023-06-25 13:38:59 +00:00 (Migrated from github.com)

The current system actually works better, so this won't be fixed.

The current system actually works better, so this won't be fixed.
LunarTides commented 2024-03-05 13:05:05 +00:00 (Migrated from github.com)

This did end up kindof being fixed

This did end up kindof being fixed
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
LunarTides/Hearthstone.js#22
No description provided.