-
Notifications
You must be signed in to change notification settings - Fork 0
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
Dokąd zmierza routes? #94
Comments
@Cosiek zaznaczyłem fajne pomysły. Edycja trasy - chyba na tym etapie za bardzo zbędna I coś, czego mi brakuje przy wyznaczaniu trasy - przycisk "usuń ostatni punkt". Powiedz mi jeszcze, czemu jeden trening może mieć wiele tras, nie na odwrót? Miałem jakiś zamysł tak robiąc, czy coś mi się pojebało? |
Wychodzi problem z brakiem dokumentacji. Kliknięcie prawym na dowolny marker, powoduje jego usunięcie.
Nie pamiętam, ale też ficzer z dzieleniem trasy przez wiele treningów nie do końca mnie przekonywał. Swoją drogą, jeśli chcesz, żeby user mógł podłączyć istniejącą trasę do treningu, to będze trzeba mu zrobić widok w którym będzie mógł to zrobić i prawdopodobnie dobrze będzie wyświetlić tam mapę z większą liczbą tras. Chyba jednak zrobię ten refactor tak, żeby umożliwić wyświetlanie wielu tras. Trochę to bardziej skomplikowane, ale coś mi mówi, że i tak będzie trzeba. Więc pod tym issue robię refactor, a potem ficzery z listy. |
👍
Ale mnie chyba przekonuje :P Samo wyświetlanie wielu tras to raczej daleka przyszłość, początkowo widzę zastosowanie dzielenie trasy na zasadzie: biegły razem dwie osoby, jedna dodała trening, druga się podpięła. Jak zrobisz refactor, to to wypchnij, wciągniemy :) |
ech, siedzę i próbuję zmienić relację tras i treningów z m2o na o2m, ale chujowo mi to idzie. boję się, że za głęboko to siedzi w tych javascriptach... |
Hmm... JS jest zupełnie nieświadomy jaka jest relacja między drogą a treningem (właściwie to nie jest świadomy istnienia treningu). Przy dodawaniu treningu zapisuje trasę AJAX-em i zwraca jej id, a przy podglądzie pobiera trasę AJAX-em na podstawie otrzymanego id. Dopóki dostaje właściwe id, nic więcej nie powinno go obchodzić. |
olałem temat, za dużo pieprzenia się... |
WIęc routes potrafi już czytać gpx-a i dodawać trasy z palca. Pytanie co jeszcze powninien umieć?
Kilka pomysłów:
Wystawiam na to issue, bo będę chciał zrefaktoryzować mapHandlera (po akceptacji #93), ale nie wiem jak. Jeżeli uznasz, że wyświetlanie wielu tras jednocześnie może nam się kiedyś przydać, to będę działał innaczej niż jeżeli stwierdzisz, że taka sytuacja nigdy nie zajdzie.
@v0y 🎱
The text was updated successfully, but these errors were encountered: