Skip to content

CST1229/zip.js -- Use correct MIME type when reading file Data.URIs - #2600

Open
SharkPool-SP wants to merge 4 commits into
masterfrom
SharkPool-SP-patch-5
Open

CST1229/zip.js -- Use correct MIME type when reading file Data.URIs#2600
SharkPool-SP wants to merge 4 commits into
masterfrom
SharkPool-SP-patch-5

Conversation

@SharkPool-SP

Copy link
Copy Markdown
Collaborator

Resolves:
#1682

Before:
image

After:
image

Not exactly a behavior-changing commit. It doesnt break compatibility since extensions such as Skins would reject the old output whereas now it will accept it.

@github-actions github-actions Bot added the pr: change existing extension Pull requests that change an existing extension label Jul 20, 2026
@SharkPool-SP

Copy link
Copy Markdown
Collaborator Author

!format

@SharkPool-SP
SharkPool-SP requested review from a team and Brackets-Coder July 20, 2026 17:17
Comment thread extensions/CST1229/zip.js Outdated
@SharkPool-SP

Copy link
Copy Markdown
Collaborator Author

!format

@Drago-Cuven

Drago-Cuven commented Jul 26, 2026

Copy link
Copy Markdown

I'd say this is a good change. Does it understand ALL file types? Or no.

(I'm hoping it understands code files like js, lua, rb, py, etc etc)

@SharkPool-SP

Copy link
Copy Markdown
Collaborator Author

I'd say this is a good change. Does it understand ALL file types? Or no.

(I'm hoping it understands code files like js, lua, rb, py, etc etc)

whichever the database supports, which includes js and lua...

@SharkPool-SP
SharkPool-SP requested a review from GarboMuffin July 27, 2026 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: change existing extension Pull requests that change an existing extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants