Skip to content

feat: Add Module:CharacterStats/Custom for omegastrikers wiki - #8012

Open
steve020607 wants to merge 2 commits into
mainfrom
CharacterStatsCustom-omegastrikers
Open

feat: Add Module:CharacterStats/Custom for omegastrikers wiki#8012
steve020607 wants to merge 2 commits into
mainfrom
CharacterStatsCustom-omegastrikers

Conversation

@steve020607

Copy link
Copy Markdown
Collaborator

Summary

same code style as #7903

to changed the StrikerStats(custom made module)

How did you test this change?

Live(tested on my testing page)
https://liquipedia.net/omegastrikers/Module:CharacterStats/Custom
https://liquipedia.net/omegastrikers/User:Steve23

@steve020607
steve020607 requested review from a team as code owners August 25, 2026 16:14
@steve020607 steve020607 changed the title feat(CharacterStats): add Custom for omegastrikers feat: Add Module:CharacterStats/Custom for omegastrikers wiki Aug 25, 2026

---@class CharacterStatsWidgetProps
---@field characterSize string
---@field characterSize string?

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.

due to the default props this is never nil

@steve020607 steve020607 Aug 25, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

But how to remove the error of previous commit?
Since there's no ?, the luals just think that it should set field on the custom version

And also most of the wiki is using this defaultProps, and nothing changes

return Array.filter(teamBans, String.isNotEmpty)
end

return OmegaStrikersCharacterStats No newline at end of file

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.

eof empty line missing

@steve020607 steve020607 Aug 26, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Do you mean adding empty line on the last line after the return OmegaStrikersCharacterStats

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants