Skip to content

Comments

Missing and corrected item images#39

Closed
SirOMGitsYOU wants to merge 6 commits intoRaidTheory:mainfrom
SirOMGitsYOU:item-image-fix
Closed

Missing and corrected item images#39
SirOMGitsYOU wants to merge 6 commits intoRaidTheory:mainfrom
SirOMGitsYOU:item-image-fix

Conversation

@SirOMGitsYOU
Copy link
Contributor

Part 1 of the breakdown of the larger PR #32.

Adds some of the missing items and fixes some incorrect textures.

@stremblayiOS
Copy link

can we merge this?

@SirOMGitsYOU
Copy link
Contributor Author

If it needs redoing due to master branch changes happy to re PR again 😂

@stremblayiOS
Copy link

you can just merge from master and fix conflicts if any

Copilot AI added a commit to Drommedhar/arcraiders-data that referenced this pull request Nov 20, 2025
- Added 40+ new item images
- Updated 3 existing item images (arc_alloy, garlic_press, plastic_parts)
- Renamed 6 item images to new naming convention (mk_X to roman numerals)

Co-authored-by: Drommedhar <11786314+Drommedhar@users.noreply.github.com>
Drommedhar added a commit to Drommedhar/arcraiders-data that referenced this pull request Nov 20, 2025
Sync item images from RaidTheory/arcraiders-data PR RaidTheory#39
@SirOMGitsYOU
Copy link
Contributor Author

you can just merge from master and fix conflicts if any

yeh just synced up the branch now.

Drommedhar added a commit to Drommedhar/arcraiders-data that referenced this pull request Nov 23, 2025
Drommedhar added a commit to Drommedhar/arcraiders-data that referenced this pull request Nov 23, 2025
Drommedhar added a commit to Drommedhar/arcraiders-data that referenced this pull request Nov 23, 2025
@baschny
Copy link
Contributor

baschny commented Jan 10, 2026

Talking with @mattmarcin about this, he proposed to use the current images he has in CDN, as these are more up to date and complete (they are being used also in arctracker.io). So I scripted to update the images in this repo with those from there, while also converting any WebP image to PNG:

#190

@mattmarcin
Copy link
Contributor

Done in #190

@mattmarcin mattmarcin closed this Jan 10, 2026
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.

4 participants