Skip to content

zadanie#1

Open
websid wants to merge 1 commit intomasterfrom
zadanieTomaszSzwed
Open

zadanie#1
websid wants to merge 1 commit intomasterfrom
zadanieTomaszSzwed

Conversation

@websid
Copy link
Copy Markdown
Owner

@websid websid commented Sep 8, 2024

  1. nie mierzyłem czasu spędzonego nad zadaniem, myślę że było to w granicach 8 godzin
  • po postawieniu dockera, miałem problem z dostępem do plików (win10+wsl2+ubuntu),
  • trochę zdziwiła mnie stara wersja php, odzwyczaiłem się od siódemki ;)
  • i ogólnie mała ilość dostępnych paczek php/js - ale w sumie rozumiem to,
  • zadanie ciekawe, dobrze opisane, nie ma się czego przyczepić
  • nigdy nie widziałem tak gołego reacta, więc niestety moje rozwiązanie też wizualnie nie powala, ale myślę, że jest wystarczająco przejrzyście, nie chciałem tracić za dużo czasu na stylowanie,
  • tworząc kod starałem się aby był czytelny i podzielony według jego odpowiedzialności,
  • listę walut zdecydowałem się trzymać w kodzie php jako consty w klasie (enum), gdyż docelowo widziałbym to pobierane z bazy danych,
  • w sumie to tak samo zrobiłem również z prowizjami i adresem endpointu nbp z którego pobierane są dane,
  1. można by ulepszyć
  • dodać testy do js
  • obsłużyć trochę więcej przypadków testowych w php
  • trochę poprawić strukturę aplikacji, poprzenosić klasy do osobnych katalogów, mały projekt to zdecydowałem się wrzucić po prostu wszystko do ogólnego Service.
  • trzeba by dopracować wygląd interfejsu użytkownika,

Chętnie porozmawiam o szczegółach implementacji podczas rozmowy technicznej (jeśli taka jest zaplanowana).

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