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

UtBS 5: changes to scenario locations #3104

Merged
merged 3 commits into from
Jun 8, 2018
Merged

UtBS 5: changes to scenario locations #3104

merged 3 commits into from
Jun 8, 2018

Conversation

UnwiseOwl
Copy link
Contributor

From #3077:

  • Moved AI leader spawns to keeps
  • Changed spawn of trolls that were spawning in walls
  • Sanity-checked village ownership in cavern

From #3078:

  • Changed spawn of Eastern Troll Flamecaster
  • Fixed entry routes of Troll Flamecasters

From author notes:

  • Made 'Wounded Dwarf' a wounded unit

From #3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

From #3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit
@Wedge009
Copy link
Member

Ditto, spaces not tabs.

@sevu sevu added High Priority Issues that cause significant usability problems and must be addressed quickly. Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign and removed High Priority Issues that cause significant usability problems and must be addressed quickly. labels May 23, 2018
[/move_unit_fake]

#TODO maybe the wounded dwarf should really be wounded?
{NAMED_NOTRAIT_UNIT 4 (Dwarvish Fighter) 21 29 (Wounded Dwarf) ( _ "Wounded Dwarf")}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the name of the dwarf now - does he get a random one?
(I'm not opposed to that)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Name is currently simply "Wounded Dwarf".

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that part is not in the code anymore - you removed it here, but it's nowhere else, only as ID.
Again, I consider this not a bad idea by default because UtBS has often long names which become too long for the sidebar if translated in other languages. But I didn't played this scenario yet.
(That's why I would prefer if someone else merges this, though looks like good to go)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies, yes you're right this now gives a random dwarf name. That was an unintentional oversight, (just when you think you've done enough testing!) though actually I think I prefer it this way as he's now evidently wounded.

@sevu sevu merged commit 7feb9ae into wesnoth:master Jun 8, 2018
sevu pushed a commit that referenced this pull request Jun 8, 2018
* UtBS 5: changes to scenario locations

From #3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

From #3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit
@UnwiseOwl UnwiseOwl deleted the subterranean branch June 12, 2018 14:50
jostephd pushed a commit to jostephd/wesnoth that referenced this pull request Oct 6, 2018
* UtBS 5: changes to scenario locations

Fixes wesnoth#3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

Fixes wesnoth#3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit
jostephd pushed a commit to jostephd/wesnoth that referenced this pull request Oct 7, 2018
* UtBS 5: changes to scenario locations

Fixes wesnoth#3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

Fixes wesnoth#3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit

(cherry-picked from commit 7feb9ae)
jostephd pushed a commit to jostephd/wesnoth that referenced this pull request Oct 18, 2018
* UtBS 5: changes to scenario locations

From wesnoth#3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

From wesnoth#3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit

(cherry picked from commit bf28cc7)
jostephd pushed a commit to jostephd/wesnoth that referenced this pull request Oct 18, 2018
* UtBS 5: changes to scenario locations

From wesnoth#3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

From wesnoth#3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit

(cherry picked from commit bf28cc7)
jostephd pushed a commit to jostephd/wesnoth that referenced this pull request Oct 27, 2018
* UtBS 5: changes to scenario locations

From wesnoth#3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

From wesnoth#3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit

(cherry picked from commit bf28cc7)
jostephd pushed a commit to jostephd/wesnoth that referenced this pull request Oct 27, 2018
* UtBS 5: changes to scenario locations

From wesnoth#3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

From wesnoth#3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit

(cherry picked from commit bf28cc7)
jostephd pushed a commit to jostephd/wesnoth that referenced this pull request Oct 28, 2018
* UtBS 5: changes to scenario locations

From wesnoth#3077:
* Moved AI leader spawns to keeps
* Changed spawn of trolls that were spawning in walls
* Sanity-checked village ownership in cavern

From wesnoth#3078:
* Changed spawn of Eastern Troll Flamecaster
* Fixed entry routes of Troll Flamecasters

From author notes:
* Made 'Wounded Dwarf' a wounded unit

(cherry picked from commit bf28cc7)
jostephd added a commit that referenced this pull request Oct 30, 2018
* josteph/forward-port-some-of-the-174:
  don't call invalidate_layout() unless needed
  Convert a bunch more C-style casts I missed in 0dc5656 to static_cast
  fixup  don't show travis notifications from forks (cherry picked from commit 55bc62f)
  don't show travis notifications from forks (cherry picked from commit d6ab780)
  fix out of bonunds check
  Lua: Invalidate layout in set_dialog_value()
  Fix includes
  Implement iOS version detection
  Game Load: Use child_or_empty().
  Themes: Fix the observers icon being hidden under the minimap
  Game Load: Restore the ability to select 1.12.6 savegames in the list
  Fix CMake build on macOS
  NR S8: compatibility commit to make the new companions from scratch if we don't have them stored
  Partially revert "Additional village variations (#3342)"
  DM: Add last breath dialog for Zorlan
  DM: Generalize death message so it works when Delfador is young and old
  gui2/unit_recall: Change Recall button back to its correct label
  Updated changelog in Xcode
  Add WML unit test for unupgradable (#3336)
  gui2/unit_recall: Fix i18n issues
  DM: add some TODOs
  iOS: finger scrolling in controls.
  iOS document directories. Might use these on other platforms as well.
  .gitignore for CLion
  German multiplayer translation: fixes
  German translation: reword era descriptions
  German translation: fix a string
  Statistics Dialog: merge unit name and count labels
  add / remove some comments
  Tutorial Part2: Give the player a canche to undo his move
  UtBS 5: changes to scenario locations (#3104)
  Update changelog in macOS package
  SotA: transform bats already after the 5th scenario
  TRoW: revert workaround for #2912
  Changelog entry for the #2912 work around
  TRoW: Increase Rithrandil's workaround portrait height to 700
  TRoW: IPF-downscale Rithrandil's portrait, as a workaround to #2912
  Add missing deprecation messages for deprecated macros
  Northern Rebirth S02_01: keep side 8 leader from wandering off too far
  changelog: remove AToTB AI entry
  wesnothd: prevent another instance of crash from owner not in player_connections
  Increased in-game chat size. This makes it render more smoothly
  Minor tweaks to classic theme (thanks enclave for doing this)
  Update RELEASE_NOTES (cherry picked from commit e2a2a85)
  Update changelog for custom AI for AToTB S2
  TRoW: revert changes, to avoid a potential OOS
  Re-apply patches to fix issues #2844 and #2846 in SotA
  Clear RELEASE_NOTES
  Repair 66ab35f in the upwards direction
  Add changelog for 1.14.0
  add placeholer links to appdata file
  Hide Editor help section
  UtBS: readd do_not_list
  UtBS: wmlindent pass
  Fix MP label for #2855
  Fixup 601c67d
  UtBS: Re-added compatibility code to load old saves using the new elves
  Swap count and name columns in statistics (fixes #2893)
  Revert string changes in SotA for 1.14 RC 3 - gold
  AI recruitment: use unit-specific recall cost
  AI recruitment: fix units on recall list interfering with recruiting
  Fixup f7ab008 before GCC complains about an unused parameter >_<
  Help: disabled parsing error popup
  LoW: Fix translation error in conjunct list (fixes #2833) (#2834)
  Fix pofix
  Changelog entry for recent SotA fixes
  Pofix entry for fc6c3ac
  Pofix entry for b86a296
  Pofix entry for ab9f29a
  Fix HP label in longer languages
  Help: removed sort_sections key with invalid value
  less redraw_everything calls when receiving [change_controller]
  fix gui2 dialog disappearing after [change_controller]
  Use a fallback for when the player name is not known in debug notifications
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants