Merge pull request 'show new notifications @ top' (#317 ) from staging into main

Reviewed-on: #317
show new notifications @ top
2024-03-28 08:11:37 +01:00 · 2024-03-28 08:10:26 +01:00
1 changed files with 1 additions and 1 deletions
--- a/src/model/notification.rs
+++ b/src/model/notification.rs
@ -58,7 +58,7 @@ impl Notification {
    pub async fn for_user(db: &SqlitePool, user: &User) -> Vec<Self> {
        sqlx::query_as!(
            Self,
-            "SELECT * FROM notification WHERE user_id = ?",
+            "SELECT * FROM notification WHERE user_id = ? ORDER BY created_at DESC",
            user.id
        )
        .fetch_all(db)
Author	SHA1	Message	Date
philipp	c5af1e4cf8	Merge pull request 'show new notifications @ top' (#317 ) from staging into main Some checks failed CI/CD Pipeline / test (push) Successful in 9m59s Details CI/CD Pipeline / deploy-staging (push) Has been skipped Details CI/CD Pipeline / deploy-main (push) Has been cancelled Details Reviewed-on: #317	2024-03-28 08:11:37 +01:00
philipp	ab565f1369	show new notifications @ top Some checks failed CI/CD Pipeline / deploy-main (push) Blocked by required conditions Details CI/CD Pipeline / test (push) Successful in 10m0s Details CI/CD Pipeline / deploy-staging (push) Has been cancelled Details	2024-03-28 08:10:26 +01:00