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

New UI, Bootstrap support and SEO optimizations #10

Merged
merged 4 commits into from
Apr 25, 2024

Conversation

Orholt
Copy link
Contributor

@Orholt Orholt commented Apr 25, 2024

Hi, I added Bootstrap 5.0 support for your UI (also I checked everything for mobile support).
Also, I added some images (Everything under Free License / Media content, so it can be used freely). I think it looks's better now, but feel free to give me your feedback.

I also updated pricing for Panek.
Furthermore, I wanted to add automatic pricing fetch for those providers, but It's just a plain JS app, so I didn't have a place for some backend magic.

Moreover, I found some spots for improvement in your project, if you have any questions feel free to contact me.

Kind regards, MW.

@pogarek
Copy link
Contributor

pogarek commented Apr 25, 2024

Wow. Thanks a lot. Any contribution or improvement is highly welcome. To be honest this my first JS "project" so, for sure, a lot of things can be improved.
I like this PR ; by the end of the weekend, I should be able to merge it.
Bardzo dziękuję!

@pogarek
Copy link
Contributor

pogarek commented Apr 25, 2024

Znalazłem, nieplanowaną, chwilę. Dziękuję. Już robię i wrzucę na stronę.
Tak, automatyzacja cennika jest do ogarnięcia. Trochę trudniej jest z pozyskaniem lokalizacji samochodów Panek, aby wrzucić je do mapki. Bo wraz z nową apką, totalnie zmienił się mechanizm logowania. Bez logowania nie mam jak pozyskać informacji o lokalizacji samochodów, zaś logowanie do Panka z poziomu tej stronki, zrobiło się skomplikowane

@pogarek pogarek merged commit ae725e8 into CSKalkulator:master Apr 25, 2024
@Orholt
Copy link
Contributor Author

Orholt commented Apr 25, 2024

@pogarek z chęcią pomogę, chociaż osobiście bym to przepisał. Apka sama w sobie super, jeśli chodzi o pomysł. Ja z racji trochę wieku, a trochę z powodu miejsca zamieszkania dość często korzystam z Traficara i naprawdę dość to ułatwia liczenie, ile wydam na podróż.

Przepisałbym dlatego, że kod nie jest dość "czysty", nie chodzi mi o samego JS, ale bardziej architekturę etc.

Nie rozumiem też użycia Dockera, ma to sens, jednak do zwykłej web-ówki dość mały.

Co do lokalizacji samochodów, jestem przekonany, że da się zrobić jakiś scrapper albo wyciągnąć API z aplikacji mobilnych, jednak na ten moment skupiłbym się osobiście na automatycznym ściąganiu ze stron cenników.

Osobiście radziłbym też rozważyć jakąś bazę danych zamiast plików .json

@pogarek
Copy link
Contributor

pogarek commented Apr 25, 2024

strona powstała bo, też, miałem dość liczenia. Zaś odkąd Panek wprowadził obowiązkowe przedpłaty, to przestałem z niego korzystać.
Przepisanie - spoko. Możesz próbować ;-)

Docker jest dla mnie, właściwie: lubię mieć swój system "czysty", więc wszystko co pisze , robię w kontenerze.

API traficara mam ogarnięte ; Panek wymaga pracy nad logowaniem skąd indziej niż strona ; do 4Mobility dawno nie zaglądałem.

Dane są w json, aby strona mogła stać na github pages i była bezkosztowa. Baza ok, o ile , także, jest bezkosztowa.
Jak wspomniałem, to mój pierwszy (po)twór w JS więc wersja 2.0 mogłaby się przydać . Ale też znaczących zmian w kodzie strony nie robiłem od dawna , wiec , sporo, po prostu, zapomniałem.

@divadsn
Copy link

divadsn commented Aug 21, 2024

Hej, odnośnie API Traficara, udostępniam bazę wraz z API gdzie też jest endpoint zwracający auta w pobliżu wybranej lokalizacji. API nie wymaga autoryzacji, CORS chyba jest wyłączony więc można je bezpośrednio użyć na stronie 😄

Dane odświeżam co minutę, w zależności jaki humor mają ich serwery danego dnia, niestety ostatnio mocno utrudniają dostęp i np bez autoryzacji zwracają fake dane dla "przykładu". Panka miałem w planach, ale przy ich obecnej sytuacji nie widzę sensu nie mieszkając w Warszawie.

@pogarek
Copy link
Contributor

pogarek commented Aug 21, 2024

Dzięki, Daawno to api traficara nie zaglądałem , aczkolwiek , kiedyś, poza Megane E-tech, wszystko zwracali bez zalogowania.

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.

3 participants