[PR #697] [MERGED] feat: sse notify #6856

New issue

Closed

opened 2026-03-01 17:14:32 +03:00 by kerem · 0 comments

kerem commented

2026-03-01 17:14:32 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/0xJacky/nginx-ui/pull/697
Author: @0xJacky
Created: 11/3/2024
Status: ✅ Merged
Merged: 11/5/2024
Merged by: @0xJacky

Base: dev ← Head: feat/sse-notify

📝 Commits (2)

e6e1876 feat: sse notify
e4a5ba5 fix: config sync issues #685

📊 Changes

20 files changed (+301 additions, -116 deletions)

View changed files

📝 api/config/get.go (+10 -8)
📝 api/config/modify.go (+18 -49)
📝 api/config/rename.go (+7 -2)
📝 api/config/router.go (+2 -2)
➕ api/notification/live.go (+31 -0)
📝 api/notification/router.go (+2 -0)
📝 app/package.json (+1 -0)
📝 app/pnpm-lock.yaml (+8 -0)
📝 app/src/api/config.ts (+2 -1)
📝 app/src/components/Notification/Notification.vue (+56 -6)
📝 app/src/components/Notification/config.ts (+4 -4)
📝 app/src/components/Notification/detailRender.ts (+41 -35)
📝 app/src/constants/index.ts (+3 -1)
📝 app/src/layouts/HeaderLayout.vue (+5 -2)
📝 app/src/views/config/ConfigEditor.vue (+2 -1)
📝 app/src/views/config/ConfigList.vue (+2 -3)
➕ app/src/views/config/components/ConfigName.vue (+67 -0)
📝 internal/config/config.go (+1 -0)
📝 internal/notification/notification.go (+38 -2)
📝 internal/site/sync.go (+1 -0)

📄 Description

Broadcast notifications via SSE

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/0xJacky/nginx-ui/pull/697 **Author:** [@0xJacky](https://github.com/0xJacky) **Created:** 11/3/2024 **Status:** ✅ Merged **Merged:** 11/5/2024 **Merged by:** [@0xJacky](https://github.com/0xJacky) **Base:** `dev` ← **Head:** `feat/sse-notify` --- ### 📝 Commits (2) - [`e6e1876`](https://github.com/0xJacky/nginx-ui/commit/e6e1876c54c10b5bb760d14cf388ea4f17e6b4af) feat: sse notify - [`e4a5ba5`](https://github.com/0xJacky/nginx-ui/commit/e4a5ba5b870d0d29674da0c9bd68abfd30500e4d) fix: config sync issues #685 ### 📊 Changes **20 files changed** (+301 additions, -116 deletions) <details> <summary>View changed files</summary> 📝 `api/config/get.go` (+10 -8) 📝 `api/config/modify.go` (+18 -49) 📝 `api/config/rename.go` (+7 -2) 📝 `api/config/router.go` (+2 -2) ➕ `api/notification/live.go` (+31 -0) 📝 `api/notification/router.go` (+2 -0) 📝 `app/package.json` (+1 -0) 📝 `app/pnpm-lock.yaml` (+8 -0) 📝 `app/src/api/config.ts` (+2 -1) 📝 `app/src/components/Notification/Notification.vue` (+56 -6) 📝 `app/src/components/Notification/config.ts` (+4 -4) 📝 `app/src/components/Notification/detailRender.ts` (+41 -35) 📝 `app/src/constants/index.ts` (+3 -1) 📝 `app/src/layouts/HeaderLayout.vue` (+5 -2) 📝 `app/src/views/config/ConfigEditor.vue` (+2 -1) 📝 `app/src/views/config/ConfigList.vue` (+2 -3) ➕ `app/src/views/config/components/ConfigName.vue` (+67 -0) 📝 `internal/config/config.go` (+1 -0) 📝 `internal/notification/notification.go` (+38 -2) 📝 `internal/site/sync.go` (+1 -0) </details> ### 📄 Description - [x] Broadcast notifications via SSE <img width="229" alt="image" src="https://github.com/user-attachments/assets/bf72d6e0-5b5b-4dd0-a91a-5b9c49fa970f"> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>