Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Yann/feature/lekaupdater/use robotkit view model #489

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

YannLocatelli
Copy link
Member

@YannLocatelli YannLocatelli commented Nov 21, 2023

Suit #464

  • Si possible, dans une autre PR utiliser les visuels de RobotKit avec la supervision de Hortense pour que ce soit cohérent visuellement
  • Bien vérifier les équivalences des classes remplacées, si des fonctions ne sont pas implémentées, les substituer après les avoir questionnées (ex. onViewAppear)

--

Comportements perdus de LekaUpdater:

  • Le scan est arrêté une fois un robot connecté et est relancé en cas de déconnexion // le scan est permanent dans RobotKit
  • Le scan est arrêté avec un cancelTask du publisher parce que le nil ne fonctionnait pas tout le temps // le scan est mis à nil dans RobotKit
  • La base de comparaison des robots des RobotDiscoveryViewModel est ID + nom // ID + charge + battery dans RobotKit
  • Lors du scan, la liste était filtré pour éviter le phénomène de multiplication du même robot // pas de filtre dans RobotKit
  • Lors du scan, le robot qui était sélectionné ou connecté reprend le même status // perte du précédent status lorsqu'un nouveau robot est détecté dans RobotKit
  • Lors de la connexion, le nom du robot est mis à jour // pas de mise à jour du nom dans Robot.shared
  • Lorsque la vue de connexion apparait, le robot précédemment connecté est (automatiquement) déconnecté // Pas de déconnexion lorsque la vue de connexion de RobotKit réapparait

Visuels perdus de LekaUpdater:

  • Absence de bouton en haut à droite // bouton Dismiss en haut à droite dans RobotKit pour skip la page de connexion
  • Arrière plan blanc uni // arrière plan bleu avec nuage dans RobotKit
  • Liste de robots sur 2 colonnes // Liste de robots sur 3 colonnes avec RobotKit -> Voir ♻️ (RobotKit): Move BackgroundView in a View #501
  • Bouton rechercher // Plus de bouton rechercher dans RobotKit
  • Bouton Se connecter actif seulement lorsqu'un robot est sélectionné // Bouton Se connecter toujours actif dans RobotKit
  • Boutons Se connecter, Se déconnecter, Continuer d'une couleur // Autre couleur dans RobotKit, non consistant avec le reste de LekaUpdater
  • Logo LekaUpdater en bas de page // N'est pas intégré dans RobotKit, dans une bande blanche dans LekaUpdater

@YannLocatelli YannLocatelli self-assigned this Nov 21, 2023
@YannLocatelli YannLocatelli force-pushed the yann/feature/lekaupdater/use-robotkit-view-model branch from e478c9c to a4921db Compare November 21, 2023 17:03
@YannLocatelli
Copy link
Member Author

IMG_0429

@ladislas ladislas force-pushed the yann/feature/lekaupdater/replace-blekit branch from cc55fad to b35f847 Compare November 21, 2023 17:14
Base automatically changed from yann/feature/lekaupdater/replace-blekit to main November 21, 2023 17:14
@YannLocatelli YannLocatelli force-pushed the yann/feature/lekaupdater/use-robotkit-view-model branch 2 times, most recently from 2cb5cf4 to 1cf9bda Compare November 27, 2023 13:38
@YannLocatelli YannLocatelli marked this pull request as ready for review November 27, 2023 14:33
@YannLocatelli YannLocatelli force-pushed the yann/feature/lekaupdater/use-robotkit-view-model branch 2 times, most recently from 12b048c to bfbd1ae Compare November 29, 2023 14:47
- delete unused files
- Remove DesignKit
- Remove VStack
- Remove viewModel init in view
- Use #Preview
@ladislas ladislas force-pushed the yann/feature/lekaupdater/use-robotkit-view-model branch from bfbd1ae to afc6e57 Compare November 29, 2023 14:50
@ladislas ladislas merged commit 75404cb into main Nov 29, 2023
5 checks passed
@ladislas ladislas deleted the yann/feature/lekaupdater/use-robotkit-view-model branch November 29, 2023 14:50
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

2 participants