Skip to content

less draconian card update function [bounty: 20 XTR] #1996

Open

Description

@DreadKnight

In the game's dash, can't see abilities of Kraken unit for example, because some energy values haven't been defined.
The function for updating that stuff should be way more lose and just skip missing stuff or have a predefined value for that.
That way, it will get easier to work on additional unit sets pretty soon, being able to constantly check out the progress.

Activity

  1. added
    codingThis issue requires some programming
    visualsVarious things that easily catch the eye
    pipelineAffects how the project is being developed
    on Dec 30, 2021
  2. added this to the 0.5 - Chimera milestone on Dec 30, 2021
  3. changed the title [-]less draconian card update function[/-] [+]less draconian card update function [bounty: 20 XTR][/+] on Oct 1, 2022
  4. added
    bountySome amount of our token, XatteR, is offered
    on Oct 2, 2022
  5. sungdark commented on Mar 26, 2026

    @sungdark

    我来认领

    收款地址:eB51DWp1uECrLZRLsE2cnyZUzfRWvzUzaJzkatTpQV9

  6. added a commit that references this issue on Mar 26, 2026
    2ef0eb9
  7. zhaog100 commented on Mar 26, 2026

    @zhaog100

    /attempt

  8. roxonn-contribution-rewards commented on Mar 26, 2026

    @roxonn-contribution-rewards

    👤 @zhaog100 is now working on this issue!

    Started: Thu, 26 Mar 2026 23:30:16 GMT

    Next steps:

    1. Create a Pull Request when ready
    2. Include fixes #1996 in PR description
    3. Add a bounty to this issue: /bounty <amount> <currency>

    Other contributors: Please coordinate in comments or work on different issues.


    Powered by Roxonn

  9. sungdark commented on Mar 27, 2026

    @sungdark

    我来认领!

    收款地址:eB51DWp1uECrLZRLsE2cnyZUzfRWvzUzaJzkatTpQV9

    I've implemented a fix in PR #2916.

    The issue was that the card update function checked costs !== undefined but then accessed costs.energy directly without verifying the energy property was defined. For units like Kraken whose abilities don't define energy costs, this caused problems.

    The fix uses optional chaining: typeof costs?.energy === 'number' to safely skip displaying cost info when it's not defined.

  10. added 2 commits that reference this issue on Mar 31, 2026
    a75c510
    6da8224
  11. domfelipe commented on Oct 30, 2026

    @domfelipe

    Claiming this one: making the card update skip missing costs instead of reading costs.energy straight off. Wallet (MintMe/XTR): 0x9f135aeB40e7B26a72AA730CF6eBb241C6b8813A

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    bountySome amount of our token, XatteR, is offeredcodingThis issue requires some programmingpipelineAffects how the project is being developedvisualsVarious things that easily catch the eye

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions