Skip to content

Conversation

@realZenith
Copy link

included are the following
->checking the deck if the goes above the standard 100 PTS and locking it to add further Point cards if you have the "Genesys" check enabled
->Adding around 105 Strings starting from 12126->12230
->Adding DUEL_MODE_GENESIS, so you can autostart into the Genesys Mode
->Added a visible notice for the total amount of Genesys PTS in your Deck, right besides the Deck Count if you have the Genesys Setting enabled
->added a duel check if you have genesys setting enabled that checks if you have to many points in deck (doesnt work properly yet)
->added LIMITATION_FILTER from 0 - 100 (only those currently in the current banlist are coded)

Currently Missing
-> Custom Point Cap for Genesys Rooms
-> idk
-> Serverside validation of a the pointlist?
-> updated strings.conf and a Genesys.lflist.conf arent included here either

I just want your Opinion for now what needs to be changed and what i have coded incorrect

including
->checking the deck if the goes above the standard 100 PTS and locking it to add further Point cards if you have the "Genesys" check enabled
->Adding around 105 Strings starting from 12126->12230
->Adding DUEL_MODE_GENESIS, so you can autostart into the Genesys Mode
->Added a visible notice for the total amount of Genesys PTS in your Deck, right besides the Deck Count if you have the Genesys Setting enabled
->added a duel check if you have genesys setting enabled that checks if you have to many points in deck (doesnt work properly yet)
->added LIMITATION_FILTER from 0 - 100 (only those currently in the current banlist are coded)

Currently Missing
-> Custom Point Cap for Genesys Rooms
-> idk
@purerosefallen
Copy link

why 100 entries

@realZenith
Copy link
Author

why 100 entries

Its not 100 Entries, its around 25~, i just choose this approach since its a unfamiliar codebase, and i had some problems setting it up. That's why I just took a simpler approach without rewriting codebase

Yes i could write it better, but i just choose this approach for simplicity lol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants