Skip to content

feat: Add module to list available factions#7816

Open
mbergen wants to merge 12 commits into
mainfrom
faction-lists
Open

feat: Add module to list available factions#7816
mbergen wants to merge 12 commits into
mainfrom
faction-lists

Conversation

@mbergen

@mbergen mbergen commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

Lists all defined factions (by game) with icons, aliases and key.

How did you test this change?

Live on https://liquipedia.net/ageofempires/Module:Faction/Data/doc (Depends on #7813)

@mbergen

This comment was marked as resolved.

@mbergen
mbergen marked this pull request as ready for review July 19, 2026 11:46
@mbergen
mbergen requested review from a team as code owners July 19, 2026 11:46

@hjpalpha hjpalpha left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  • should use widget v3 instead of v2
  • for wikis that don't have game specific versions (i.e. all but aoe) this won't work well, maybe add a (bool) option input param |byGame=true for that? (if set to false use default game without box, if set to true use Game.listGames)

@mbergen

mbergen commented Jul 19, 2026

Copy link
Copy Markdown
Collaborator Author
* for wikis that don't have game specific versions (i.e. all but aoe) this won't work well, maybe add a (bool) option input param `|byGame=true` for that? (if set to false use default game without box, if set to true use Game.listGames)

Works perfectly fine on wikis that have only a single game.
grafik

Now SC2 is different as you configure different games (versions)?
In the end, would you even want to use it on sc2?
grafik

Comment thread lua/wikis/commons/Faction/Lists.lua Outdated
Comment thread lua/wikis/commons/Faction/Lists.lua Outdated
Comment thread lua/wikis/commons/Faction/Lists.lua Outdated
mbergen and others added 3 commits July 19, 2026 18:14
Co-authored-by: hjpalpha <75081997+hjpalpha@users.noreply.github.com>
Co-authored-by: hjpalpha <75081997+hjpalpha@users.noreply.github.com>
Co-authored-by: hjpalpha <75081997+hjpalpha@users.noreply.github.com>
Comment thread lua/wikis/commons/Faction/Lists.lua
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants