2023-07-24 Code Review

⚠️ Pirms kaut ko labo paņem merge no MASTER branch, jo es jau izmainīju daļu styles!

 

  1. Iznest atsevišķā Component header un turpmāk daļas, kuras var re-use citos Screens uzreiz iznest kā components (šis būs vajadzīgs jaunajā control screen) image-20230724104521435 image-20230724104531807

 

  1. image-20230724104636303

 

  1. Jaunajā mapping screen nedrīkst būt šādi image-20230724104908538

Tas notiek, jo nav implementēts Throttling, parāk daudz ROS events fonā saukti, es pievienoju, bet turpmāk šim jāseko līdzi

Es pievienoju @babel/plugin-proposal-decorators tāpēc tev vajag palaist npm i, vairāki dependencies nomainījās

 

  1. image-20230724105126071

 

 

  1. image-20230724105201402

 

  1. image-20230724105251733

 

  1. image-20230724105518592

 

  1. Šo es salaboju bet AIZLIEGTS subscribe to events un tad NEUZTAISIT unsubscribe!!! LIELA PROBLEMA. Tu nebiji salicis unsubscribe. Izskatās, ka daļai iepriekšējo topics arī nebija pielikts, kas ir nepareizi, pieliku image-20230724110136657

 

  1. kapēc šis nebija implementēts

    image-20230724112426125

 

  1. Tu nebiji kārtīgi izpētījis messages saturu image-20230724123134759

image-20230724123200401Var redzēt, ka tie nav radians! Ignored messages, kurus vajadzēja apstrādāt!

Praktiski UI šādi nevar - vajag pa vienam message rādīt image-20230724123708005