mpfmorawski / git-to-know-me

3 stars 1 forks source link

Agregacja danych do podstawowych funkcjonalności #16

Closed mpfmorawski closed 2 years ago

mpfmorawski commented 2 years ago

Elementy do dodania do obecnego kodu bez dodawania nowych funkcjonalności:

Nowe funkcjonalności do dodania:

Przydatne linki:

jprokopczuk commented 2 years ago

W celu uproszczenia i przyspieszenia prac nad kodem, ograniczamy się tylko do repozytoriów użytkownika.

jprokopczuk commented 2 years ago

Wraz z @kamilwil postanowiliśmy trochę uprościć sposób odczytywania danych z repozytoriów. Główne uproszczenia to:

Proszę odwołajcie się do tego @mpfmorawski i @pktiuk. @kamilwil potwierdź czy dobrze się zrozumieliśmy :).

kamilwil commented 2 years ago

Potwierdzam, dobrze się zrozumieliśmy :)

jprokopczuk commented 2 years ago

Kolejne uproszczenie. TOP 5 repozytoriów przesyłanych przez endpoint /api/stats/top_repos (a tym samym też endpoint /api/stats/languages) to tak na prawdę 5 pierwszych repozytoriów.

@mpfmorawski , @pktiuk

Dobrze by było, aby ta funkcjonalność przetestował ktoś z liczbą publicznych repozytoriów większą niż 5 :). @pktiuk

mpfmorawski commented 2 years ago

5 pierwszych repozytoriów pod jakim względem? Czy mógłbyś rozwinąć tę myśl?

jprokopczuk commented 2 years ago

Po względem kolejności :). To była poprawka na szybko aby dzisiaj móc już łączyć kod z frontendem.

Pierwsze 5 repozytoriów, które dostanę z endpointa users/{user}/repos (API GitHuba).

mpfmorawski commented 2 years ago

Żeby już połączyć to z frontendem to tymczasowo ok. Ale do ostatecznej wersji aplikacji nie dopuszczam takiego druciarstwa

jprokopczuk commented 2 years ago

Dobra, poprawie...

jprokopczuk commented 2 years ago

@mpfmorawski, druciarstwo poprawione na trochę mniejsze druciarstwo. Teraz repozytoria sortują się po liczbie gwiazdek.