nonobis/Heimdall

Watch

Fork 0

feat(cameras): fusionner les pages Caméras et Détection en un seul écran #167

Merged

nonobis merged 1 commit from feat/feat-fusionner-les-crans-cam-ras-et-d-tection-d-un into main

2026-08-26 17:24:15 +02:00

nonobis commented

2026-08-26 16:50:42 +02:00

Owner

Contexte

Depuis la page d'un site (SiteDetail), les écrans « Caméras » et « Détection » étaient séparés avec des tuiles distinctes, créant une confusion pour l'utilisateur. Les deux écrans se recouvraient pour gérer les caméras d'un site.

Cette PR fusionne les deux dans une page unique avec trois onglets MudTabs :

Caméras : liste, ajout manuel, suppression, snapshots
Découverte : scan réseau, résultats, confirmation d'ajout
Identifiants : gestion des credentials caméras

Tickets liés

Fixes #109

Tests

1436 tests unitaires passent (Domain + Application + Web + Infrastructure + Agent)
Tests DiscoveriesPageTests réécrits pour tester via l'onglet Découverte de la page fusionnée
Tests CamerasPageTests et SiteDetailTests mis à jour
Aucune régression détectée

Notes

Rétrocompatibilité : /sites/{id}/discoveries redirige vers /sites/{id}/cameras (DiscoveriesRedirect.razor)
SiteDetail : la tuile « Détection » est supprimée, la tuile « Caméras » affiche « Gérer les caméras, détection et identifiants »
Ancienne DiscoveriesPage supprimée : le contenu est migré dans l'onglet Découverte de CamerasPage
Fix pré-existant : UpdateNotificationConfigRequest.SubscribedEvents manquant dans HeimdallApiClientCoverageTests.cs
Commit non signé : pas de clé GPG disponible sur l'hôte d'agent

Maquette

Lien Penpot : page sites

## Contexte Depuis la page d'un site (SiteDetail), les écrans « Caméras » et « Détection » étaient séparés avec des tuiles distinctes, créant une confusion pour l'utilisateur. Les deux écrans se recouvraient pour gérer les caméras d'un site. Cette PR fusionne les deux dans une page unique avec trois onglets MudTabs : - **Caméras** : liste, ajout manuel, suppression, snapshots - **Découverte** : scan réseau, résultats, confirmation d'ajout - **Identifiants** : gestion des credentials caméras ## Tickets liés - Fixes #109 ## Tests - 1436 tests unitaires passent (Domain + Application + Web + Infrastructure + Agent) - Tests DiscoveriesPageTests réécrits pour tester via l'onglet Découverte de la page fusionnée - Tests CamerasPageTests et SiteDetailTests mis à jour - Aucune régression détectée ## Notes - **Rétrocompatibilité** : `/sites/{id}/discoveries` redirige vers `/sites/{id}/cameras` (DiscoveriesRedirect.razor) - **SiteDetail** : la tuile « Détection » est supprimée, la tuile « Caméras » affiche « Gérer les caméras, détection et identifiants » - **Ancienne DiscoveriesPage supprimée** : le contenu est migré dans l'onglet Découverte de CamerasPage - **Fix pré-existant** : `UpdateNotificationConfigRequest.SubscribedEvents` manquant dans `HeimdallApiClientCoverageTests.cs` - **Commit non signé** : pas de clé GPG disponible sur l'hôte d'agent ## Maquette Lien Penpot : [page sites](https://penpot.tordais.cc/6f49870c-7da0-8185-8008-7a1ed728877b?page=127dfd1c-12b5-80f7-8008-79c6fda854a1)

nonobis added 1 commit

2026-08-26 16:50:42 +02:00

feat(cameras): fusionner les pages Caméras et Détection en un seul écran

CI / Docker + Trivy (pull_request) Has been skipped

Details

CI / Détection changements (pull_request) Successful in 9s

Details

CI / OWASP Dependency-Check (pull_request) Has been skipped

Details

CI / Docker PR (pull_request) Successful in 3m6s

Details

Auto-Fix / Auto-Fix (pull_request) Successful in 10m34s

Details

CI / Build + Format (pull_request) Successful in 14m9s

Details

CI / Tests DB (Infrastructure) (pull_request) Has been skipped

Details

CI / Tests Web (bUnit) (pull_request) Successful in 3m26s

Details

CI / Tests unitaires (pull_request) Successful in 3m26s

Details

CI / Résumé tests (PR) (pull_request) Successful in 4s

Details

CI / SonarQube (pull_request) Successful in 7m13s

Details

CI / ci (pull_request) Successful in 1s

Details

b2ed2e107a

Depuis la page d'un site, les écrans « Caméras » et « Détection »
étaient séparés avec des tuiles distinctes, créant une confusion
pour l'utilisateur. Cette PR fusionne les deux dans une page unique
avec trois onglets : Caméras (liste+ajout+suppression),
Découverte (scan réseau+confirmation) et Identifiants (credentials).

- CamerasPage : onglets MudTabs avec Caméras, Découverte, Identifiants
- SiteDetail : suppression de la tuile « Détection », 2 tuiles au lieu de 3
- DiscoveriesRedirect : redirection /discoveries → /cameras (rétrocompat)
- Ancienne DiscoveriesPage supprimée (contenu migré dans CamerasPage)
- Tests DiscoveriesPageTests réécrits pour tester via l'onglet Découverte
- Fix: UpdateNotificationConfigRequest.SubscribedEvents manquant dans les tests

Closes #109

nonobis commented

2026-08-26 17:01:06 +02:00

Author

Owner

🤖 Coding Orchestrator — 2026-08-26 15:01 UTC

Changement détecté: code

auto-fix-trivial

dotnet format OK
⏭️ À vérifier

review-gate

OK
⏭️ À vérifier

Ce commentaire est généré automatiquement par le coding orchestrator.

## 🤖 Coding Orchestrator — 2026-08-26 15:01 UTC **Changement détecté:** code ### auto-fix-trivial - dotnet format OK **⏭️ À vérifier** ### review-gate - OK **⏭️ À vérifier** --- _Ce commentaire est généré automatiquement par le coding orchestrator._

nonobis commented

2026-08-26 17:07:00 +02:00

Author

Owner

🤖 Coding Orchestrator — 2026-08-26 15:07 UTC

Changement détecté: code

auto-fix-trivial

dotnet format OK
⏭️ À vérifier

review-gate

OK
⏭️ À vérifier

Ce commentaire est généré automatiquement par le coding orchestrator.

## 🤖 Coding Orchestrator — 2026-08-26 15:07 UTC **Changement détecté:** code ### auto-fix-trivial - dotnet format OK **⏭️ À vérifier** ### review-gate - OK **⏭️ À vérifier** --- _Ce commentaire est généré automatiquement par le coding orchestrator._

forgejo-actions commented

2026-08-26 17:08:28 +02:00

🤖 Résumé des tests

Tests unitaires

Projet	Statut	Passed	Total
`Heimdall.Agent.Tests`	🟢	299	299
`Heimdall.Api.Tests`	🟢	2	2
`Heimdall.Application.Tests`	🟢	468	468
`Heimdall.Domain.Tests`	🟢	168	168

✅ 937 tests passent.

Tests Web (bUnit)

Projet	Statut	Passed	Failed	Skipped	Total
`Heimdall.Web.Tests`	🟢	319	0	0	319

✅ 319 tests passent.

Tests DB (Infrastructure)

Skippé (aucun changement DB/Domain)

Généré automatiquement par la CI.

## 🤖 Résumé des tests ## Tests unitaires | Projet | Statut | Passed | Failed | Skipped | Total | |---|---|---|---|---|---| | `Heimdall.Agent.Tests` | 🟢 | 299 | 0 | 0 | 299 | | `Heimdall.Api.Tests` | 🟢 | 2 | 0 | 0 | 2 | | `Heimdall.Application.Tests` | 🟢 | 468 | 0 | 0 | 468 | | `Heimdall.Domain.Tests` | 🟢 | 168 | 0 | 0 | 168 | **✅ 937 tests passent.** --- ## Tests Web (bUnit) | Projet | Statut | Passed | Failed | Skipped | Total | |---|---|---|---|---|---| | `Heimdall.Web.Tests` | 🟢 | 319 | 0 | 0 | 319 | **✅ 319 tests passent.** --- ## Tests DB (Infrastructure) _Skippé (aucun changement DB/Domain)_ _Généré automatiquement par la CI._

nonobis commented

2026-08-26 17:16:21 +02:00

Author

Owner

🤖 Coding Orchestrator — 2026-08-26 15:16 UTC

Changement détecté: code

auto-fix-trivial

dotnet format OK
⏭️ À vérifier

review-gate

OK
⏭️ À vérifier

Ce commentaire est généré automatiquement par le coding orchestrator.

## 🤖 Coding Orchestrator — 2026-08-26 15:16 UTC **Changement détecté:** code ### auto-fix-trivial - dotnet format OK **⏭️ À vérifier** ### review-gate - OK **⏭️ À vérifier** --- _Ce commentaire est généré automatiquement par le coding orchestrator._

nonobis merged commit 3a9bc48326 into main

2026-08-26 17:24:15 +02:00

nonobis deleted branch feat/feat-fusionner-les-crans-cam-ras-et-d-tection-d-un

2026-08-26 17:24:15 +02:00

nonobis referenced this pull request from a commit

2026-08-26 17:24:16 +02:00

feat(cameras): fusionner les pages Caméras et Détection en un seul écran (#167)