Skip to content

Feature/11167: Proper labelling of boss-enemies#11169

Merged
tool4ever merged 1 commit into
Card-Forge:masterfrom
BramTeurlings:feature/archipelago/corrected-enemy-data
Jul 7, 2026
Merged

Feature/11167: Proper labelling of boss-enemies#11169
tool4ever merged 1 commit into
Card-Forge:masterfrom
BramTeurlings:feature/archipelago/corrected-enemy-data

Conversation

@BramTeurlings

@BramTeurlings BramTeurlings commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Split off from: Archipelago PR

Split between "bosses" and "minibosses"

  • The main bosses: Lorthos, Emrakul, Lathliss, Ghalta, Griselbrand & Akroma are currently indistinguishable from mini-bosses in the game files. This is something that I would like to have for the Archipelago implentation so I suggest moving the aforementioned 6 inside of the decks\boss folder and all of the other bosses inside of the decks\miniboss folder.
  • Various bosses were wrongly labeled as not being a boss, Jace comes to mind. Each enemy that has a unique boss effect on the battlefield is now labeled properly as "boss" in Shandalar's enemies.json, their deck is also moved to the appropriate folder as mentioned in the previous point.
  • Some enemies had the "boss" label in Shandalar's enemies.json while not being a boss at all, the boss label has been removed from these enemies.
  • Originally, the split between these two folders already existed but it seems that someone moved all bosses into one decks\boss folder in the last 6 months. This can still be seen in forge-gui/res/adventure/Shandalar Old Border/world/enemies.json as many of them still reference decks\miniboss instead of boss. This PR will make the regular Shandalar game consistent with Old Border again.

@Hanmac Hanmac requested a review from kevlahnota July 6, 2026 15:11
@Hanmac Hanmac added the Adventure For the Adventure Module label Jul 6, 2026
@Hanmac Hanmac requested review from Simisays and removed request for kevlahnota July 6, 2026 15:12
@tool4ever tool4ever merged commit 16951c9 into Card-Forge:master Jul 7, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Adventure For the Adventure Module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants