Project

General

Profile

Feature #4768

Implementering af DDB Cover servicens upload funktion (AFVENTER #4864)

Added by Rolf Madsen 5 months ago. Updated about 1 month ago.

Status:
Resolved (tag version)
Priority:
Urgent
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Søgning - Værkvisning, Søgning - Materialevisning, Søgning - Søgeresultat efter søg - Brønd

Description

Problemstilling

Bibliotekerne mangler forsidebilleder der ikke findes i DDB Cover servicen, fordi de ikke ikke findes i de kilder vi importerer fra, eller ikke kan matches automatisk til de tilhørende katalogposter på grund af manglende ID eller metadata.

Formål

Giv mulighed for manuelt at uploade billeder og matche dem til katalogposter.

Løsningsforslag

Jesper do your magic! :-)

History

#1 Updated by Kasper Garnæs 3 months ago

  • Status changed from Ready for development to Reviewed - Needs info/rework

Jeg har reviewet PR'et her: https://github.com/ding2/ding2/pull/1617

#2 Updated by Jesper Kristensen 3 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Jesper Kristensen to Kasper Garnæs

Så er det lavet ændringer fra DDB test, UX test og review.

#3 Updated by Kasper Garnæs 3 months ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Jesper Kristensen

Jeg har reviewed og har stadig et par enkelte punkter.

Hvis vi skal vende det på et Skype-møde e.l. så sig til.

#4 Updated by Gitte Barlach 2 months ago

  • Assignee changed from Jesper Kristensen to Jørgen Nielsen

Hej Jørgen
Vil du give dit besyv med her?

#5 Updated by Jesper Kristensen 2 months ago

Det du skal kigge på er navn givningen af provider kald til at få tokens etc.

#6 Updated by Jørgen Nielsen 2 months ago

  • Assignee changed from Jørgen Nielsen to Jesper Kristensen

jeg har gjort mit bedste for øge forvirringen ;-)

#7 Updated by Gitte Barlach 2 months ago

  • Priority changed from Normal to Urgent

#8 Updated by Jesper Kristensen 2 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Jesper Kristensen to Kasper Garnæs

Klar til review igen

#9 Updated by Kasper Garnæs 2 months ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Jesper Pedersen Pedersen

Jeg har rereviewed. Jeg har kun enkelte kommentarer vedr. CSS tilbage.

Jesper: Hvis vi skal vende det på et Skypemøde eller lignende så sig til.

#10 Updated by Kasper Garnæs 2 months ago

  • Status changed from Reviewed - Needs info/rework to Reviewed
  • Assignee changed from Jesper Pedersen Pedersen to Kasper Garnæs

Jesper har opdateret PR'et. Tak for det.

Reviewed og godkendt.

 

#11 Updated by Kasper Garnæs about 2 months ago

  • Status changed from Reviewed to Technical test

Merged.

#12 Updated by Kasper Garnæs about 2 months ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Jesper Kristensen

Ændringerne i dette PR [fejler i vores CI proces](https://circleci.com/gh/aakb/ding2/2613). Jeg [mistænkte fejlen at være urelateret til ændringerne](https://github.com/ding2/ding2/pull/1617#pullrequestreview-427969370) men jeg kan konstatere at [fejlen også opstår når ændringerne merges til master branchen](https://circleci.com/gh/ding2/ding2/8120). Derfor har jeg været nødt til at [rulle PR'et tilbage i master branchen](https://github.com/ding2/ding2/pull/1643).

Jesper: Vil du undersøge problemet nærmere? Hvis jeg kan hjælpe gør jeg gerne det.

#13 Updated by Jesper Kristensen about 2 months ago

Lavet revert af revert PR https://github.com/ding2/ding2/pull/1645 og arbejde på problemet

#14 Updated by Jesper Kristensen about 2 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Jesper Kristensen to Kasper Garnæs

Så er det nye PR gået i grøn :-)

PR: https://github.com/ding2/ding2/pull/1645

#15 Updated by Kasper Garnæs about 2 months ago

  • Status changed from Needs code review to Technical test

Godkendt og merged.

#16 Updated by Kasper Garnæs about 2 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#17 Updated by Gitte Barlach about 2 months ago

  • Subject changed from Implementering af DDB Cover servicens upload funktion to Implementering af DDB Cover servicens upload funktion (AFVENTER #4864)

Jeg tester denne når vi ellers er klar.

#18 Updated by Gitte Barlach about 1 month ago

  • Status changed from Technical test to Resolved (tag version)

Testet på upgrade-fbs med 7.x-6.1.0-beta5


- trin 1-4 af upload med udgangspunkt i hhv. alm søgning samt søgning via /admin/config/ting/ddb_cover_upload - ok, godkendt
- oversigt over uploadede forsider /admin/config/ting/ddb_cover_upload/list  - ok, godkendt
- sletning af uploaded forside - ok, godkendt
- preview Step 1 - ok, godkendt
- brugsbetingelser Step 3 - tekst + link - ok, godkendt
- de uploadede covers er synlige på de respektive poster efter upload - ok, og godkendt

Hermed vil jeg samlet set godkende sagen. 

Also available in: Atom PDF