Skip to content

Alert user to packages of concern#2074

Open
ebkr wants to merge 15 commits into
developfrom
feature/unlisted-notifications/alert-vulnerable-packages
Open

Alert user to packages of concern#2074
ebkr wants to merge 15 commits into
developfrom
feature/unlisted-notifications/alert-vulnerable-packages

Conversation

@ebkr

@ebkr ebkr commented Feb 13, 2026

Copy link
Copy Markdown
Owner

Alerting users to packages of concern

Identifies mods no longer available on Thunderstore and gives the user a choice on how to proceed

How does it work

Based on the tracking in #2071

Mods can be taken down for several reasons, and this informs the user when it has happened.

This implementation adds a new kind of UX flow where clicking the review banner will now filter the local mod list. This is an optional step but makes the identified packages easier to discover. There is now a single tag that can be visibly applied, and auto-deselects once all packages have been resolved.

A package can be reviewed by expanding and clicking on the "Review package" button within the notification box. The user is then asked to either mark a package as safe, or to remove the mod.

Screenshots

Identified package in regular mod list

image

Narrowed scope

image

Searching is disabled when scope is narrowed

image

Expanded content

image

Review modal

image

Marked as safe

image

Light theme examples

image image image

@ebkr ebkr requested a review from x753 February 13, 2026 11:07
@ebkr ebkr force-pushed the feature/unlisted-notifications/tracking branch from a9deede to 24486d5 Compare February 16, 2026 15:16
@ebkr ebkr force-pushed the feature/unlisted-notifications/alert-vulnerable-packages branch from 829ddf3 to df92b23 Compare February 16, 2026 15:21

@x753 x753 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Didn't build and test, but here's a review of the code and content.

Also, I think the out of sync icon should appear for all local mods + mods no longer found on Thunderstore that were marked safe, otherwise people can't tell at a glance what isn't being exported with their profiles. I think we discussed the export profile modal showing unexported mods, but the out of sync icon could probably be addressed through this PR.

Comment thread src/components/modals/ConcerningPackageReviewModal.vue Outdated
Comment thread src/components/views/LocalModList/LocalModCard.vue Outdated
Comment thread src/components/banner/ConcerningPackageBanner.vue Outdated
Comment thread src/components/views/LocalModList/LocalModCard.vue Outdated
@Marioalexsan

Copy link
Copy Markdown
Contributor

Seems to me that this is linked to #1988 and #1989, so those should be closed if this is merged.

@ebkr

ebkr commented May 6, 2026

Copy link
Copy Markdown
Owner Author

After addressed changes:
image

image image image

@x753

x753 commented May 6, 2026

Copy link
Copy Markdown
Collaborator

Looks better.
image
What's this for though?

@ebkr ebkr requested a review from x753 May 7, 2026 12:36
@x753

x753 commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Gave it a test. "Unlinked" should apply to any mods that aren't connected to Thunderstore, including locally installed mods, and some sort of warning should appear when exporting a profile with any unlinked mods.

Versions that can't be found on Thunderstore also don't seem to appear as unlinked despite them technically being so.

Base automatically changed from feature/unlisted-notifications/tracking to develop May 13, 2026 10:26
@ebkr ebkr force-pushed the feature/unlisted-notifications/alert-vulnerable-packages branch from e3a57af to 826bcbf Compare May 13, 2026 11:26
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.

3 participants