Skip to content

Conversation

@batstyx
Copy link
Contributor

@batstyx batstyx commented Mar 16, 2023

Resolves #68
Resolves #64
Closes #63
Resolves #62
Resolves #53

batstyx and others added 30 commits December 8, 2021 17:29
Even build numbering on master
Displays adjusted Cost but also uses Count=Cost to get around card image caching
To match 'Race' for minions
- Replaces Individual view isValid functions
- Moves view enabled setting checks to use ViewConfig
- Adds view to panel
LastPlayedView definitely not working
Title is a bit messy?
Add SinglesPanel for single entry views
joc256 and others added 19 commits March 30, 2025 14:50
- main.yml
  - Rename to `Build+Release` from `Build+Release on PR`
  - Switch trigger from `pull_request` to `push`
- build+upload.yml
  - Copy main.yml content minus "create_release" step (retaining separate build versioning)
  - Upload zip instead of just dll
And remove unused 7z generated zip
And remove double zip extension
Reference AnimatedCards collection instead of Items collection from removed ItemsControl base
Graveyard UI not showing: MissingMethodException AnimatedCardList.Update
Just to note that ChatGPT *did not* help, at all
@batstyx
Copy link
Contributor Author

batstyx commented May 17, 2025

I hadn't meant to leave this pull request open quite this long, so apologies for the 250+ commits.
Finally stopped playing last year so haven't added any new features since Whizbang's Workshop.
All updates past that are bug fixes and compatibility updates.

Change log highlights

  • GitHub Workflows
    • Build + Upload on PR
    • Build + Release on Push
  • AnyfinCalculator removed as I couldn't get it to calculate accurately without major re-write, Murloc card counter retained
  • EventManager as API GameEvents intermediary
  • Settings.Upgrade to enable updates to Settings file format
  • Plugin.Graveyard settings folder to allow for Dropbox/OneDrive based setting sharing
  • Re-written Settings UI to allow for card list-based view enabling
  • Refactoring to allow view initialisation based on settings card list

Understand entirely if no-one wants to deal with this lot.
Me doing something about it was triggered by HearthSim/Hearthstone-Deck-Tracker#4627 being raised now that I'm in [minor] maintenance mode.

@batstyx batstyx marked this pull request as ready for review May 17, 2025 17:27
batstyx added 9 commits July 3, 2025 08:52
- Soft removed with QuestlineVisibility setting
- FriendlyQuestlineEnabled value and default set to False
- EnemyQuestlineEnabled value and default set to False
- Closes #17

Can be re-enabled by adding
```
<Setting>
  <Name>QuestlineVisibility</Name>
  <Value>Visible</Value>
</Setting>
```
to `%appdata%\HearthstoneDeckTracker\Plugin.Graveyard\Settings.xml`
Supports multiple card versions, where a card name has more than one id associated with it
e.g. `Tess Greymane` in **The Witchwood** and **Core**
- Enable/Disable all card-based view triggers
- Resurrection/Deathrattle views shown based only on player 'Resurrect' and 'Deathrattle' settings
**Little Things**
- [Questline no longer supported](b577973)
- [Card List "Reset to Default" button](8d8ed64)
- [Compare cards for view activation by name rather than id](d89b3a5)
- [Player 'Cards' setting](db7aee1)
- HearthstoneDeckTracker.exe to v1.46.6.7098
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants