Skip to content

Kapi Koin tracker#85

Merged
ProjectInfinity merged 2 commits into
classic-addon-manager:masterfrom
CraySone:master
May 25, 2026
Merged

Kapi Koin tracker#85
ProjectInfinity merged 2 commits into
classic-addon-manager:masterfrom
CraySone:master

Conversation

@CraySone
Copy link
Copy Markdown
Contributor

@CraySone CraySone commented May 25, 2026

Kapi Koin tracker

Copilot AI review requested due to automatic review settings May 25, 2026 08:20
@ProjectInfinity
Copy link
Copy Markdown
Contributor

ProjectInfinity commented May 25, 2026

Looks good to me, but one question: you have a icon file that looks like a logo but since it is not named "icon.png" it will not be displayed in the addon manager. Is it intentional to not show it in the addon manager? If so I can merge as is, but if you want it shown you should rename the file and update usage (so it still works in-game).

Let me know.

@CraySone
Copy link
Copy Markdown
Contributor Author

CraySone commented May 25, 2026

Looks good to me, but one question: you have a icon file that looks like a logo but since it is not named "icon.png" it will not be displayed in the addon manager. Is it intentional to not show it in the addon manager? If so I can merge as is, but if you want it shown you should rename the file and update usage (so it still works in-game).

Let me know.

Its the icon for ingame UI button, so you can just merge!

@ProjectInfinity ProjectInfinity merged commit b7e59f4 into classic-addon-manager:master May 25, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Awaiting requested review from Copilot

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@CraySone @ProjectInfinity