[PR #372] [MERGED] 365 add alerts and errors to requests in new UI #402

New issue

Closed

opened 2026-02-27 19:18:03 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 19:18:03 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/EddieTheCubeHead/Stagnum/pull/372
Author: @EddieTheCubeHead
Created: 6/28/2024
Status: ✅ Merged
Merged: 6/28/2024
Merged by: @EddieTheCubeHead

Base: master ← Head: 365-add-alerts-and-errors-to-requests-in-new-ui

📝 Commits (6)

bfc3c20 #365 Basic framework for alerts. Finally typed methods.ts somewhat properly
652b851 #365 Improvements to alert system and styling
52eddd3 #365 Added testing for alerts and successes
6bb853a #365 Improve avatar error resistance, provide mock functions for throwing exceptions in axios requests
b29d183 #365 Add tests for happy cases, improve success alert messages
3357aa1 #365 Fix overwriting with the new data model

📊 Changes

32 files changed (+362 additions, -75 deletions)

View changed files

📝 clientRewrite/setup-vitest.ts (+4 -0)
➕ clientRewrite/src/alertSystem/Alert.ts (+9 -0)
➕ clientRewrite/src/alertSystem/AlertComponent.tsx (+29 -0)
➕ clientRewrite/src/alertSystem/AlertHandler.tsx (+18 -0)
➕ clientRewrite/src/alertSystem/alertStore.ts (+22 -0)
📝 clientRewrite/src/api/methods.ts (+34 -14)
📝 clientRewrite/src/common/components/avatar/Avatar.tsx (+2 -2)
📝 clientRewrite/src/common/components/toolbar/ToolBar.tsx (+1 -1)
📝 clientRewrite/src/common/hooks/useMeQuery.ts (+1 -1)
📝 clientRewrite/src/common/stores/poolStore.ts (+5 -4)
📝 clientRewrite/src/common/views/Home.tsx (+3 -1)
📝 clientRewrite/src/globals.css (+4 -4)
📝 clientRewrite/src/pool/components/ConfirmPoolOverwriteModal.tsx (+6 -3)
📝 clientRewrite/src/pool/components/PoolMemberCard.tsx (+1 -1)
📝 clientRewrite/src/pool/components/SharePool.tsx (+3 -2)
📝 clientRewrite/src/pool/hooks/useDeletePool.ts (+8 -4)
📝 clientRewrite/src/pool/hooks/useDeletePoolContent.ts (+9 -4)
📝 clientRewrite/src/pool/hooks/useJoinPool.ts (+2 -1)
📝 clientRewrite/src/search/components/SearchTracks.tsx (+1 -1)
📝 clientRewrite/src/search/components/cards/SearchResourceCard.tsx (+3 -3)

...and 12 more files

📄 Description

No description provided

_{🔄 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/EddieTheCubeHead/Stagnum/pull/372 **Author:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Created:** 6/28/2024 **Status:** ✅ Merged **Merged:** 6/28/2024 **Merged by:** [@EddieTheCubeHead](https://github.com/EddieTheCubeHead) **Base:** `master` ← **Head:** `365-add-alerts-and-errors-to-requests-in-new-ui` --- ### 📝 Commits (6) - [`bfc3c20`](https://github.com/EddieTheCubeHead/Stagnum/commit/bfc3c200c1a0f89eddd654a9ca8c301e17937a52) #365 Basic framework for alerts. Finally typed methods.ts somewhat properly - [`652b851`](https://github.com/EddieTheCubeHead/Stagnum/commit/652b851d104dda85cbdc6138bca937b76cc7043d) #365 Improvements to alert system and styling - [`52eddd3`](https://github.com/EddieTheCubeHead/Stagnum/commit/52eddd383bfebe1bf326c810d92feedc0c0b40d2) #365 Added testing for alerts and successes - [`6bb853a`](https://github.com/EddieTheCubeHead/Stagnum/commit/6bb853a7f761ba73d7548b919398be16f1d5690c) #365 Improve avatar error resistance, provide mock functions for throwing exceptions in axios requests - [`b29d183`](https://github.com/EddieTheCubeHead/Stagnum/commit/b29d183338e970feb8d102006bda55cabda02324) #365 Add tests for happy cases, improve success alert messages - [`3357aa1`](https://github.com/EddieTheCubeHead/Stagnum/commit/3357aa1e4f7bae6b70f9306a30ab0d9bb69b21e0) #365 Fix overwriting with the new data model ### 📊 Changes **32 files changed** (+362 additions, -75 deletions) <details> <summary>View changed files</summary> 📝 `clientRewrite/setup-vitest.ts` (+4 -0) ➕ `clientRewrite/src/alertSystem/Alert.ts` (+9 -0) ➕ `clientRewrite/src/alertSystem/AlertComponent.tsx` (+29 -0) ➕ `clientRewrite/src/alertSystem/AlertHandler.tsx` (+18 -0) ➕ `clientRewrite/src/alertSystem/alertStore.ts` (+22 -0) 📝 `clientRewrite/src/api/methods.ts` (+34 -14) 📝 `clientRewrite/src/common/components/avatar/Avatar.tsx` (+2 -2) 📝 `clientRewrite/src/common/components/toolbar/ToolBar.tsx` (+1 -1) 📝 `clientRewrite/src/common/hooks/useMeQuery.ts` (+1 -1) 📝 `clientRewrite/src/common/stores/poolStore.ts` (+5 -4) 📝 `clientRewrite/src/common/views/Home.tsx` (+3 -1) 📝 `clientRewrite/src/globals.css` (+4 -4) 📝 `clientRewrite/src/pool/components/ConfirmPoolOverwriteModal.tsx` (+6 -3) 📝 `clientRewrite/src/pool/components/PoolMemberCard.tsx` (+1 -1) 📝 `clientRewrite/src/pool/components/SharePool.tsx` (+3 -2) 📝 `clientRewrite/src/pool/hooks/useDeletePool.ts` (+8 -4) 📝 `clientRewrite/src/pool/hooks/useDeletePoolContent.ts` (+9 -4) 📝 `clientRewrite/src/pool/hooks/useJoinPool.ts` (+2 -1) 📝 `clientRewrite/src/search/components/SearchTracks.tsx` (+1 -1) 📝 `clientRewrite/src/search/components/cards/SearchResourceCard.tsx` (+3 -3) _...and 12 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>