Opened 10 years ago

Closed 5 years ago

Last modified 5 years ago

#2631 closed enhancement (fixed)

Make the wall_builder.js rmgen library civ agnostic (possibly using the skirmish replacer)

Reported by: Niek Owned by:
Priority: Should Have Milestone: Alpha 23
Component: Maps Keywords:
Cc: Patch:

Description

Currently it is a pain to make every civ compatible with the Wall Placement in the Random Map Generation Library, since you have to add it manually to wall_builder.js. It would be better to pick the skirmish replacements instead.

Change History (7)

comment:1 by Niek, 10 years ago

Component: Core engine → UI & Simulation
Milestone: Backlog → Alpha 17
Type: defect → enhancement

comment:2 by Itms, 10 years ago

Milestone: Alpha 17 → Alpha 18

comment:3 by Itms, 9 years ago

Milestone: Alpha 18 → Alpha 19

comment:4 by mimo, 9 years ago

Milestone: Alpha 19 → Backlog

comment:5 by FeXoR, 9 years ago

It's worked on in #2944 I don't know how the skirmish replacer works and if that could help solving this.

Last edited 9 years ago by FeXoR (previous) (diff)

comment:6 by s0600204, 5 years ago

Component: UI & Simulation → Maps
Milestone: Backlog → Alpha 23
Resolution: → fixed
Status: new → closed

As of r20625 (​rP20625) it is no longer necessary to modify the wall_builder.js file to support custom or new civilisations.

As such, the core issue of this ticket is resolved (even if not by the proposed method).

comment:7 by elexis, 5 years ago

Summary: Let the RMGEN-library pick the skirmish replacer instead of a fixed list in wall_builder.js → Make the wall_builder.js rmgen library civ agnostic (possibly using the skirmish replacer)
Note: See TracTickets for help on using tickets.