prepare code for yse-siirrin; clean code; fixes partially #200

kouralex commented 4 years ago

@miguelahonen I accidentally PR'd this to master instead of development, so one must revert the usual code flow after merge (or delete & re-PR to correct branch).

kouralex commented 4 years ago

Make a reference to the relevant issue: #200

miguelvaara commented 4 years ago

Käydäänkö tätä yhdessä läpi, kun @kouralex olet taas mukana "remmissä". Miten testasit tehdyt muutokset ja niiden seurannaisvaikutukset käytännössä?

miguelvaara commented 4 years ago

@kouralex, koodi toimi oikein ja @miguelahonen unohti testissään yhden ehdon (yse_term). Kuitenkin päädyttiin siihen, että suggestion/open/skos-endpointia pitää muutta seuraavasti:

in suggestions.py:

muutetaan funktion nimi universaalimmaksi ja tarvittaessa refaktoroidaan
yhdistetään et_open_suggestions_skos() ja get_suggestion_skos()
tuloksena syntyy muokattu universaalimpi ja universaalimmaksi nimetty endpoint, jossa otetaan vastaan query-string-parametri, jonka kutsuja välittää urlin kautta.
query-stringistä parsitaan tieto siitä, -> mitkä ehdouksen staten ovat inklusiivisia tai eksklusiivisia -> onko mukana YSEssä -> mikä on palautettavan paketin muoto (oletus: skos) -> mikä on palautettavan paketin formaatti (mandatory: turtle)

Alustava aikatauluarvio on 2-3 päivää

NatLibFi / Finto-suggestions

prepare code for yse-siirrin; clean code; fixes partially #200 #274