Repository navigation
Add XML to all classes in the Card folder, add GainMultipleGems ability class - #343
Open
HumabHatterZed wants to merge 42 commits into
Conversation
**Comment out unusued duplicate field**
Considering if the custom costs ID system can be deprecated, but it may have a use as a way of providing custom cost adjustments on CardModInfos
… not using it except in very specific cases
…verse key-value pair of currentSacrificedCardInfos, add XML
… two gems" sigils floating about
Chaosyr
requested changes
Nov 13, 2026
Chaosyr
left a comment
Chaosyr
left a comment
Collaborator
There was a problem hiding this comment.
If its of use, the following contains common XML tags that can be used and how to use them. https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/xmldoc/recommended-tags
This PR is good once the two items mentioned are fixed across the module ^^
Add some missing XML and fix incorrect XML in AbilityExtensions
Chaosyr
requested changes
Nov 13, 2026
Chaosyr
left a comment
Chaosyr
left a comment
Collaborator
There was a problem hiding this comment.
Just fix these areas, otherwise looks good thus far ^^
…rterDeckSelectScreenPatches class name
Chaosyr
approved these changes
Nov 14, 2026
Chaosyr
left a comment
Chaosyr
left a comment
Collaborator
There was a problem hiding this comment.
Gonna wait till im given the go ahead to merge but this looks good ^^
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds all missing XML to classes contained in the API's Card subfolder.
Also adds GainMultipleGems, an ability subclass that allows for modders to create sigils that provide multiple gem colours instead of just one or all three.