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.

Sign up for GitHub

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

Jump to bottom

Přidání zpáteční cesty #2413

Merged
merged 10 commits into from
Nov 5, 2023
Merged

Přidání zpáteční cesty #2413

merged 10 commits into from
Nov 5, 2023

Conversation

jan-stanek
Copy link
Member

closes #2265

@jan-stanek jan-stanek marked this pull request as draft October 25, 2023 20:35
@jan-stanek jan-stanek marked this pull request as ready for review October 25, 2023 22:07
@sinacek
Copy link
Member

sinacek commented Oct 26, 2023

Nelíbí se mi moc ta textace Obrátit cestu. Je to spíš něco jako Zkopírovat s obrácením směru

@jan-stanek
Copy link
Member Author

Dělá to jen obrácení aktualní cesty, kopii je třeba vytvořit předem

@sinacek
Copy link
Member

sinacek commented Oct 26, 2023

A proč to dělat na 2 kroky, když je to jediny use case?

@jan-stanek
Copy link
Member Author

Takhle mi to přijde univerzálnější, ale máš pravdu, že "přidat zpáteční trasu" bude asi nejčastější a mělo by to stačit

@jan-stanek
Copy link
Member Author

jan-stanek commented Oct 27, 2023 •
edited
Loading

Co tam mít obojí? Tím, jak se nedá měnit pořadí cest v rámci dne, tak bych to osobně zadával asi tak, že si naduplikuju hromadu cest a pak je budu postupně otáčet, abych nemusel střídavě duplikovat cestu tam a zpátky. Ale chápu, že jen pro přidání zpáteční cesty je to krok navíc.

@sinacek
Copy link
Member

sinacek commented Oct 29, 2023

A co kdyby tam bylo jen jedno tlacitko duplikovat & otočit, neboli Zpopírovat jako zpáteční? to můžeš klikat 10x a rovnou ti to udělá co potřrebuješ bez nutnosti klikat 2x.

@jan-stanek jan-stanek changed the title Obrácení směru cesty Přidání zpáteční cesty Oct 31, 2023
@jan-stanek
Copy link
Member Author

Upraveno, nebo myslíš dát pryč i to tlačítko na duplikování?

@jan-stanek jan-stanek force-pushed the reverse-travel branch 2 times, most recently from 00d382e to 4bbc3a4 Compare October 31, 2023 21:00
@sinacek
Copy link
Member

sinacek commented Nov 1, 2023

Upraveno, nebo myslíš dát pryč i to tlačítko na duplikování?

tak to asi zkusme takto

@jan-stanek
Copy link
Member Author

Padající testy předpokládám souvisí s tou změnou localhost url, tak pak udělám rebase, až to bude opravené...

@jan-stanek
Copy link
Member Author

rebased, zamerguješ prosím?

@sinacek sinacek merged commit 9962bf0 into skaut:master Nov 5, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sinacek sinacek sinacek approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Tlačítko pro prohození startu a cíle v cesťáku
2 participants
@jan-stanek @sinacek