Skip to content

Add notifications for incoming mail - #4817

Open
RogueVoidling wants to merge 4 commits into
new-frontiers-14:masterfrom
RogueVoidling:mail-notification
Open

Add notifications for incoming mail#4817
RogueVoidling wants to merge 4 commits into
new-frontiers-14:masterfrom
RogueVoidling:mail-notification

Conversation

@RogueVoidling

Copy link
Copy Markdown
Contributor

About the PR

Mail carriers(technically, anyone with a MailMetrics cartridge installed on their PDA) will now receive notifications whenever teleporters warp a batch of mail in.

Why / Balance

I just thought it would be nice for mail carriers to know when mail was warped in, so they could know when they needed to fly back and pick more up. I also added a button to toggle the notification so SRs don't have to get spammed with it, and carriers can turn it off at their preference.

Technical details

Changes to the MailMetrics: Added a new field that controls if notifications are on or off, defaulting to on. This is passed to the client whenever the state of the mail stats is updated. The server will also now listen for an event indicating notification preference has been changed, and upon receiving it, toggle the server side variable. The client side UI will also display this value, and send a message whenever its toggled.
Changes to MailSystem: Added a method that pushes a notification to everyone with the MailMetrics installed, and notifications enabled.

How to test

Spawn in a MailMetrics cartridge and install it
Use the mailnow command to spawn in mail, notice you get a notification
Toggle the notifications on and off, using the button in MailMetrics, while using mailnow to ensure the toggle works
(Remember to clear the mail teleporter so more mail can spawn!)

Media

Requirements

Breaking changes

Changelog

🆑

  • add: Mail carriers will now receive a toggleable PDA notification whenever mail enters the sector.

@github-actions github-actions Bot added size/M S: Untriaged S: Needs Review This PR is awaiting reviews C# UI FTL and removed S: Needs Review This PR is awaiting reviews S: Untriaged size/M labels Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant