Skip to content

Feature/zadanie_maciej_jas#58

Open
maciejjas02 wants to merge 2 commits intotelemedico:masterfrom
maciejjas02:feature/zadanie-Maciej_Jas
Open

Feature/zadanie_maciej_jas#58
maciejjas02 wants to merge 2 commits intotelemedico:masterfrom
maciejjas02:feature/zadanie-Maciej_Jas

Conversation

@maciejjas02
Copy link
Copy Markdown

@maciejjas02 maciejjas02 commented Oct 26, 2025

  1. 3 dni po 3/4h
  2. API jest bardzo stabilne i zadanie odzwierciedla realne potrzeby biznesowe
  3. W backendzie miałem na względzie podział odpowiedzialności
  • NbpClient dla API,
  • RateCalculator dla logiki marż,
  • RatesService jako organizer

Jeśli chodzi o front to początkowo próbowałem Webpack, ale przeszedłem na React CDN dla prostoty. Chart.js + React sprawiał problemy, więc zastosowałem bridge funkcje vanilla JS.

56 testów w piramidzie - unit dla logiki, integration dla kontrolerów, functional end-to-end. Mocki NBP dla przewidywalnych danych.

Korzystałem z dokumentacji Symfony, NBP API, Stack Overflow oraz GitHub Copilot dla przyspieszenia pisania boilerplate kodu i testów. Szukałem też rozwiązań na różnych forach.

Chciałem też postawić na moją innowację w przypadku samego wyglądu strony więc postawiłem na animacje tła i napisów.

…ór daty; serwisy NBP + kalkulator; sprzątnięcie starego DefaultController
- Backend Symfony PHP 8.2 z integracj NBP API
- Frontend React + Vanilla JS z wykresami Chart.js
- Obsuga 5 walut z poprawnymi marami kantowymi
- 56 testów pokrywajcych ca funkcjonalno
- Docker setup gotowy do produkcji na porcie 80
- Automatyczne przekierowanie i wydzielony CSS
- Wszystkie komentarze po polsku, profesjonalna dokumentacja
@maciejjas02 maciejjas02 force-pushed the feature/zadanie-Maciej_Jas branch from defdf12 to c8a1ebe Compare October 26, 2025 19:05
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.

1 participant