Skip to content

fix: ignore extensions for mrpack warning modal - #6948

Open
IMB11 wants to merge 1 commit into
mainfrom
cal/ignore-extensions-mrpack-warning-modal
Open

fix: ignore extensions for mrpack warning modal#6948
IMB11 wants to merge 1 commit into
mainfrom
cal/ignore-extensions-mrpack-warning-modal

Conversation

@IMB11

@IMB11 IMB11 commented Jul 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Member
  • Fixes the broken ignore extensions logic, broke with shared instances.

@modrinth-bot

modrinth-bot commented Jul 31, 2026 •
edited by IMB11
Loading

Copy link
Copy Markdown
Member

Pull request changelog

App

Added

Changed

  • Common config file formats and RPO files are now hidden from the external modpack file warning modal.

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@IMB11 IMB11 added the frontend Involves work from the frontend team label Aug 3, 2026
@Prospector

Prospector commented Aug 4, 2026

Copy link
Copy Markdown
Member

@Mergifyio queue

@mergify

mergify Bot commented Aug 4, 2026

Copy link
Copy Markdown

Merge Queue Status

  • 🟠 Waiting for queue conditions
  • ⏳ Enter queue
  • ⏳ Run checks
  • ⏳ Merge
Waiting for
  • -conflict [📌 queue requirement]
All conditions
  • -conflict [📌 queue requirement]
  • -closed [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of [📌 queue -> configuration change requirements]:
    • -mergify-configuration-changed
    • check-success = Configuration changed
  • any of [📌 queue requirement]:
    • check-neutral = Mergify Merge Protections
    • check-skipped = Mergify Merge Protections
    • check-success = Mergify Merge Protections
  • any of [🔀 queue conditions]:
    • all of [📌 queue conditions of queue rule default]:
      • github-review-approved [🛡 GitHub branch protection]
      • any of [🛡 GitHub branch protection]:
        • check-success = Lint and Test
        • check-neutral = Lint and Test
        • check-skipped = Lint and Test

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

Reviewers

@Prospector Prospector Prospector approved these changes

Assignees

No one assigned

Labels

frontend Involves work from the frontend team

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@IMB11 @modrinth-bot @Prospector