Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add null check for PlayerSideboardsDict to fix #38 #40

Closed
wants to merge 114 commits into from

Conversation

joc256
Copy link

@joc256 joc256 commented Jan 4, 2025

#38

I don't know C# so I'm pretty much trusting Copilot for this...

batstyx added 30 commits May 6, 2020 13:57
Add 'Rummaging Kobold' to 'Weapons Used'
Simple Lady Liadrin view
Guldan -> Demons view
Demons views includes Kanrethad Ebonlocke
Original zh-CN translation by ZelKnow26
Fixed intermittent card miscount bug
Simple Lady Liadrin view
Guldan -> Demons view
Demons views includes Kanrethad Ebonlocke
Original zh-CN translation by ZelKnow26
Fixed intermittent card miscount bug
- Vectus added to Deathrattle view
- Graveyard Opacity and Scale sliders updated
 - Scale slider changed from 0-100 to 50-200
 - Opacity slider changed from 0-100 to 20-100
 - Current value text added for all sliders
- Save on property change removed
- Rally! view
Limit Rally! to 1-3 cost minions
Rally! view cost range correction
Use card EnglishText for Corrupted and Frenzy checks
Fix Orientation settings load error
- Grand Magus Antonidas
- Elwynn Boar
Forged in the Barrens
- Kargal Battlescar
batstyx and others added 29 commits June 12, 2023 15:47
  - [Warrior] Power Slider
  - [Neutral] The One Amalgam Band
* Mostly accurate

Update build and release workflow
Shows the list of cards in the 'snapshot'
Will show the snapshot of the last Fizzle played.
Works for serial, but not parallel, snapshots.
Serial ✓: Fizzle->Snapshot->Fizzle->Snapshot
Parallel ✗:Fizzle->Fizzle->Snapshot->Snapshot
- "Last Spell by School" view
  - [Mage] Sif added
  - [Mage] Inquisitive Creation added
- [Paladin] Tyr basic chance view added
- [Paladin] Tyr's Tears chance view added
- "Last Spell by School" view
  - [Mage] Wisdom of Norgannon added
  - [Mage] Elemental Inspiration added
- "Minions Created" view added
  - [Priest] Ra-den added
  - Lists minions created *not* played
- % chance removed from Tyr's Tears view as not accurate for either card version (unforged/forged)
- Plugin Settings Folder
Graveyard.xml -> Plugin.Graveyard\Settings.xml
- Convert Position settings from pixels to percentages

Fixes
Update CardId references
Set to original set *not* to Wonders version
- Paladin
  - Silvermoon Portal
  - Lay on Hands
- Priest
  - Onyx Bishop
- Neutral
  -Street Trickster
- Add Cultivation view
- Update "Mulchmuncher" condition for "Blood Treant"
This Should be the Body

Getting git head and splitting it:-
- title
- body
Should be Body
Extra Body
- Item 1
- Item 2

Body after a space
Should be Body
# Conflicts:
#	.github/workflows/main.yml
Error introduced in HDT 1.22.9, cause unclear
Fixes #25
- HDT to v1.22.9.6344
- Issues #25 #26
- Write build message to md file
- Help link url tooltips
- Add [Mage] Unchained Guardian to Grand Finale view
- Azerite Rat view added
- Kingpin Pud view added
- Player/Opponent Location #29
- Opponent Location Reset #30
- Player Display Reset #31
- List excluding Neutral minions RedHatter#32
- Improved horizontal alignment #28
- Endgame [Warlock] last demon death view
- Joymaster Jepetto view added
  - 1 Health or 1 Attack minions listed
- Lady Liadrin spell list updated
  - Blessings of Kings reverted to legacy id
- From: Hand default order (chronological?)
- To: Hand zone position order
- See #35
- Include E.T.C. Band in view activation card list
- Check hand/deck for more Fizzle Snapshots before clearing list
- List all 5+ cost mechs that have died
- Temporary Zilliax Deluxe 3000 name workaround, assumes any variants used cost 5+
- HearthstoneDeckTracker.exe v1.36.2.6905
- HearthDb.dll v31.0.0
- #37
@joc256 joc256 closed this Jan 4, 2025
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