Skip to content

feat: Add GameHistory to GitHub - #8088

Open
steve020607 wants to merge 4 commits into
mainfrom
GameHistory
Open

steve020607 wants to merge 4 commits into
mainfrom
GameHistory

Conversation

@steve020607

Copy link
Copy Markdown
Collaborator

Summary

This PR is changing the local wiki Game history to the Standardized GameHistory

This Code is developed by Claude and me

How did you test this change?

dev

@steve020607
steve020607 requested review from a team as code owners September 14, 2026 15:26
@steve020607 steve020607 self-assigned this Sep 14, 2026

@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.

this barely uses GameTable
why is it building a display itself instead of utilizing GameTable properly?

local inputs = self:_readOpponentInputsFromBase(base)

if Logic.isEmpty(inputs) then
assert(self.title.namespace == 0, 'Required ' .. base .. '= argument')

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.

don't remove this assert
you need to bypass this case entirely for your purposes
the table insert below it is wrong for your case too

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.

I remember this, if i remove it, it turn to an error if |team= are not inputed

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.

yeah, but inserting basepagename as team is wrong for your case too, hence the remark that you have to bypass the entire if block for your case

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.

So what is the fix for this?

Comment thread lua/wikis/commons/MatchTable.lua Outdated
@steve020607

Copy link
Copy Markdown
Collaborator Author

this barely uses GameTable
why is it building a display itself instead of utilizing GameTable properly?

So of it are override the GameTable since to fix some of the issue that GameTable didn't had

Or you mean by importing it to the GameTable and use it there?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants