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

Reakce na testovací verzi #626

Merged
merged 5 commits into from
May 5, 2022
Merged

Reakce na testovací verzi #626

merged 5 commits into from
May 5, 2022

Conversation

Fasand
Copy link
Contributor

@Fasand Fasand commented May 3, 2022

prošli jsme ten test, kromě toho, co psal Luboš je tady pár věcí:

  • v sekci Tematické kolekce (https://www.webarchiv.cz/cs/tematicke-kolekce) se zobrazuje údaj o zobrazení (vizuální/textové) vlevo, v ostré verzi napravo
  • na webu nefunguje katalog stránek, nezobrazují se screenshoty a nefunguje recaptcha, což je přepokládám ok, že se natáhnou až v ostré verzi
  • Vizuální/textové se nyní zobrazuje vpravo, stejně jak to je na produkci (byl to zbytek z migrace Bootstrapu)
  • Katalog stránek a další padaly, protože screenshoty jsou v databázi jakoby zadané, ale soubory neexistují. Upravil jsem to, aby to před načtením thumbnailu zkontrolovalu, jestli soubor doopravdy existuje, nyní se tedy zobrazí jen "WWW" placeholder, ale stránka se načte.
  • ReCaptcha tam samozřejmě nyní není nastavena, na produkci by ale měla jet. V configuraci jsou proměnné "RECAPTCHA_PUBLIC_KEY" a "RECAPTCHA_PRIVATE_KEY", k těm já nemám přístup a pro test ani nejsou potřeba.
  • fix interní tem. kolekce - úprava názvu pole "semínka podle podle frekvence" #612 : frekvenční pole na tématických kolekcích se nyní jmenuje pouze "Frekvence", což dává sémanticky větší smysl
  • partial Specifikace API #599 : nová API dokumentace jak v doce/new_docs/api.md tak na https://github.com/WebarchivCZ/Seeder/wiki/API-Documentation

@mariehaskovcova
Copy link
Contributor

díky, ty změny, které jsi udělal, nejsou vidět na tom nejnovějším testu, viď? otestovali bychom frekvenci

@Fasand
Copy link
Contributor Author

Fasand commented May 5, 2022

Jj, ještě to tam není. Rychle sepíšu něco k tomu API a nahodím to

… w/ DRF's Schema generator – API also on Github Wiki
@Fasand Fasand marked this pull request as ready for review May 5, 2022 09:27
@Fasand Fasand merged commit 881ccee into master May 5, 2022
@Fasand Fasand deleted the changes branch May 5, 2022 09:27
@Fasand
Copy link
Contributor Author

Fasand commented May 5, 2022

@mariehaskovcova
Copy link
Contributor

díky, test zkoukneme. Díky za API dokumentaci, poprosila bych pak Petru @habetpet (po návratu do práce) o její pohled na dokumentaci, díky

@mariehaskovcova
Copy link
Contributor

koukali jsme mj. na tu frekvenci, vypadá to dobře (plánování sklizní tem. kolekcí podle frekvence), viz #612, díky, už je to uživatelsky srozumitelný

plánování sklizní jako takových bude dávat smysl po vyřešení scénářů v issue #611

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.

interní tem. kolekce - úprava názvu pole "semínka podle podle frekvence"
2 participants