Skip to content

Add notifications#3045

Merged
omfj merged 10 commits into
mainfrom
omfj/notifications
May 2, 2026
Merged

Add notifications#3045
omfj merged 10 commits into
mainfrom
omfj/notifications

Conversation

@omfj

@omfj omfj commented May 1, 2026

Copy link
Copy Markdown
Member
  • feat: add notification table
  • feat: implement notifications in uno
  • feat: add notifications to uno client
  • refactor: add more context to failed fetch
  • feat: add notifications popover
  • feat: add patch to allowed methods
  • feat: add reply notification
  • fix: docs name
  • fix: use slug instead of _id for comment section identifiers

@omfj
omfj force-pushed the omfj/notifications branch from f4b2fbe to 9e70044 Compare May 1, 2026 12:09
@omfj
omfj force-pushed the omfj/notifications branch from 9e70044 to 8a9fd55 Compare May 1, 2026 12:11
@omfj
omfj requested review from 0Zeno and jesperkha May 1, 2026 12:12
@omfj
omfj force-pushed the omfj/notifications branch from 8a9fd55 to ae3f603 Compare May 1, 2026 12:13
@github-actions

github-actions Bot commented May 1, 2026

Copy link
Copy Markdown

Preview Environment

Project URL
Frontend https://pr-3045.web.echo-webkom.no
Backend https://pr-3045.uno.echo-webkom.no
Status 🟢 Deployed

Deployed from 52f3e33 at 2.5.2026, 19:58:29

Comment thread apps/uno/domain/service/notification.go Outdated
Comment thread apps/uno/domain/service/notification.go Outdated
Comment thread apps/uno/http/handler/context.go Outdated
Comment thread apps/uno/http/routes/api/comment.go Outdated
Comment thread apps/uno/http/routes/api/notification.go Outdated
Comment thread apps/uno/infrastructure/postgres/comment.go Outdated
Comment thread apps/uno/infrastructure/postgres/notification.go Outdated
@omfj
omfj force-pushed the omfj/notifications branch from ae3f603 to a95e630 Compare May 1, 2026 13:06
@omfj
omfj force-pushed the omfj/notifications branch from a95e630 to 325f34b Compare May 1, 2026 13:11
0Zeno
0Zeno previously approved these changes May 2, 2026

@0Zeno 0Zeno left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Kanskje legge til noen notifikasjoner i staging miljøet også?

@omfj

omfj commented May 2, 2026

Copy link
Copy Markdown
Member Author

Test notifications?

@omfj

omfj commented May 2, 2026

Copy link
Copy Markdown
Member Author

Kan legge det til i developer tools. Lager issue på det

@omfj
omfj enabled auto-merge (squash) May 2, 2026 18:22
@omfj
omfj disabled auto-merge May 2, 2026 18:40
@omfj
omfj merged commit f4d2aab into main May 2, 2026
15 checks passed
@omfj
omfj deleted the omfj/notifications branch May 2, 2026 18:40
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