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

Ekstrakcja stanu wszystkich baz naraz prosto ze strony głównej profilu #36

Open
3 tasks
Noiredd opened this issue May 31, 2020 · 1 comment
Open
3 tasks
Labels
enhancement New feature or request important Features that will be worked on
Milestone

Comments

@Noiredd
Copy link
Owner

Noiredd commented May 31, 2020

Teraz każda baza z osobna musi odpytać API jaki jest stan odpowiadającej jej bazy zdalnej. FilmwebAPI robi to przez sprawdzenie pierwszej strony dla każdej z baz.

Informacja ta jest jednak dostępna bezpośrednio na stronie głównej profilu użytkownika. Można by zatem wykonać takie zapytanie za jednym zamachem dla wszystkich baz naraz. API mogłoby to robić w tle, a poszczególne bazy pytając API o swoje stany zdalne (również w wątkach, czemu nie) już nie wykonywałyby kilku dodatkowych żądań tylko otrzymywały wyniki z cache'u.

TODO:

  • odnaleźć, gdzie w dokumencie przechowywane są te informacje (prawdopodobnie JSON)
  • oprogramować ekstrakcję tych informacji
  • zaimplementować dla nich cache i przerobić FilmwebAPI.getNumOf by zeń korzystał

A w przyszłości także:

  • przerobić FilmwebAPI na działanie w tle (zewnętrzne obiekty zlecają requesty, które API realizuje w wątku, a one - w swoich wątkach - oczekują na rezultaty, np. otrzymując unikatowy token, po którym mogą pytać API czy żądany zasób jest już dostępny),
  • przerobić Database na działanie w tle, by aktualizacja wywoływana była automatycznie w osobnym wątku,
  • zabezpieczyć API i żądające obiekty na wypadek utraty połączenia i innych problemów (możliwość bezpiecznego anulowania żądania).
@Noiredd Noiredd added enhancement New feature or request important Features that will be worked on labels May 31, 2020
@Noiredd Noiredd added this to the v1.0.0 milestone Jun 4, 2020
@Noiredd
Copy link
Owner Author

Noiredd commented Apr 21, 2022

Update: strona profilu wyświetla teraz błędne informacje dotyczące "chcę zobaczyć" - po wejściu w stronę listy liczba jest inna. Możliwe, że cała informacja jest przekazywana do strony głównej, a ona tylko prezentuje co trzeba (do sprawdzenia), ale w przeciwnym razie to nie ma sensu.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request important Features that will be worked on
Projects
None yet
Development

No branches or pull requests

1 participant