Card-Forge / Card-Forge/forge

OptionalCostValue updates needed

Open
#1,815 4 comments 0 reactions 1 assignee Claimed by @Hanmac View on GitHub
Game Mechanics keep
Dominant language
Java
Stars
2.7k
Forks
1.1k
Avg merge
1d 1h
Merged PRs (30d)
163

Description

OptionalCostValue currently stores Type and Cost, it need to also store staticAbilityId with default either null or -1

isOptionalCostPaid could be extended to check for the staticAbilityId too

then when OptionalCost static is checked in CostAdjustment, it could check for if this specific ability was used

_Originally posted by @Hanmac in https://github.com/Card-Forge/forge/pull/1759#discussion_r1010197642_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.