Skip to content

Conversation

@rviscomi
Copy link
Collaborator

@rviscomi rviscomi commented Oct 3, 2025

Progress on #3417

In the Matrix chat @captainbrosset said:

Ah, I think you're right, that we should just change "low" to "high" in this feature.
It would have been good to break the build here. I thought we had sanity checks for these kinds of things.
I actually came across a test just yesterday that, I think, is supposed to check for dates and baseline stages ...
But I'll need @ddbeck to investigate more deeply here

This PR addresses the immediate issue of manually updating the feature's status to "high" but there's still some investigation needed into why it didn't get flagged sooner.

Updated the Baseline status for notifications from low to high and add the corresponding baseline_high_date
@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label Oct 3, 2025
Copy link
Contributor

@captainbrosset captainbrosset left a comment

Choose a reason for hiding this comment

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

LGTM

@rviscomi rviscomi merged commit fa59e0b into main Oct 3, 2025
4 checks passed
@rviscomi rviscomi deleted the notifications-apps-high branch October 3, 2025 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature definition Creating or defining new features or groups of features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants