mi2-warsaw / CzasDojazdu

https://hub.docker.com/r/marcinkosinski/czasdojazdu/builds/
5 stars 1 forks source link

Reakcja na przycisk "Pokaż lokalizacje" #36

Open krzyslom opened 7 years ago

krzyslom commented 7 years ago

Po pierwszym kliknięciu na przycisk aplikacja staje się czuła na każdą zmianę dokonaną w inpucie. W związku z tym przy każdym missclicku związanym z datą czy przedziałem cenowym zaciągane są dane. Czy są jakieś przeciwwskazania, aby aplikacja była czuła tylko na kolejne wciśnięcia "Pokaż lokalizacje"?

Dodatkowo, Google API wykorzystywane jest dla zadanego przedziału dat. Niestety za każdym razem, kiedy dla tego samego przedziału dat zmieniana jest cena lub środek transportu, dokładnie te same dane zaciągane są ponownie. Najlepiej jakby dane z API ściągane były raz dla zadanego przedziału dat, następnie przy zmianie innych atrybutów aplikacja bazowała na już ściągniętych danych. W wypadku zwiększenia przedziału czasowego powinny zostać dociągnięte tylko te dane o które zwiększył się przedział.

MarcinKosinski commented 7 years ago

Niegłupie, ale myśle ze poruszasz tu 2 watki, wiec warto to rozbić na 2 zadania.

Co do tego guzika 'Pokaz lokalizacje' to przecież jest to 'Action Button' wiec aplikacja powinna robić cokolwiek dopiero po jego kliknięciu (nawet za n-tym razem). Chyba ze w najnowszym shiny cos sie zmieniło w tej kwestii, ja sie zatrzymałem na wersji 0.14