Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update skins.xml #593

Merged
merged 3 commits into from
Feb 8, 2025
Merged

Update skins.xml #593

merged 3 commits into from
Feb 8, 2025

Conversation

SpeedyFolf
Copy link
Contributor

@SpeedyFolf SpeedyFolf commented Jan 6, 2025

Standardized skins.xml files for admin, admin2, editor_gui, and freeroam.

Added missing skins to admin2 and editor_gui, removed duplicate 103 (Baller Jacket) entries, and removed space and trailing comma from keywords.

@SpeedyFolf SpeedyFolf requested a review from jlillis as a code owner January 6, 2025 15:02
@Kinimel
Copy link

Kinimel commented Jan 7, 2025

I agree you have a good point that it could be a problem of missing skins point of view and they should be reintegrated into the skins panel and not be missing from the skins panel

Copy link
Contributor

@Nico8340 Nico8340 left a comment

Choose a reason for hiding this comment

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

Good

@Dutchman101 Dutchman101 merged commit 5ac6fb3 into multitheftauto:master Feb 8, 2025
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants