Skip to content

Conversation

@AlliotTech
Copy link
Contributor

@AlliotTech AlliotTech commented Jan 5, 2026

  • Add a “Push Message” action and dialog on the Messages page
    (only for app-specific views)

  • Send messages using the application token, avoiding
    interceptor overwrite

  • Add UI tests covering button visibility and send flow

  • Add quick filters to the Messages page (search title/body
    and exact priority)

Preview

Google Chrome 2026-01-05 14 46 42 Google Chrome 2026-01-05 14 38 38 image

Related issues

#276

@AlliotTech AlliotTech requested a review from a team as a code owner January 5, 2026 06:51
@AlliotTech AlliotTech changed the title feat: Add push-message UI on Messages page with tests feat: Add push-message UI on Messages page with tests && feat: Add message list filtering and in-app push UI Jan 5, 2026
@AlliotTech AlliotTech mentioned this pull request Jan 5, 2026
@AlliotTech
Copy link
Contributor Author

I'm sorry to hear that.

after reviewing all the issues related to the search feature, it's clear from the comments and reactions that many people are interested in this functionality. (In this PR, I only implemented a UI filter feature, avoiding any intrusive backend complexity.)

if someone is still interested in this filter function, here is the branch reference: https://github.com/AlliotTech/server/tree/feat/ui-search

Anyway, I have already removed the search functionality from this PR. The current PR now only includes the push message feature.

Copy link
Member

@jmattheis jmattheis left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution. I've added some remarks in subcomments.

@AlliotTech AlliotTech requested a review from jmattheis January 7, 2026 15:35
@AlliotTech
Copy link
Contributor Author

Thank you, I've already fixed it.

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.15%. Comparing base (0315db4) to head (6e432e1).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #897   +/-   ##
=======================================
  Coverage   79.15%   79.15%           
=======================================
  Files          56       56           
  Lines        2226     2226           
=======================================
  Hits         1762     1762           
  Misses        360      360           
  Partials      104      104           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@jmattheis jmattheis left a comment

Choose a reason for hiding this comment

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

Thanks.

@jmattheis jmattheis merged commit bbf7483 into gotify:master Jan 8, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants