Project

General

Profile

Bug #4039

Reservering udløser fejl ("søge-cirkel" i det uendelige)

Added by Tue Gaston over 1 year ago. Updated about 1 year ago.

Status:
Resolved (tag version)
Priority:
Urgent
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Søgning - Materialevisning

Description

Jeg er logget ind på https://upgrade-fbs.ddbcms.dk med testbruger Caroline Conrad DDB

Jeg fremfinder et materiale ("Den moralske Don Juan", faustnr 20767405) 

Jeg klikker på reservér, og får dette skærmbillede:

 

Og dén forsvinder ikke, heller ikke efter et kvarter.

Ved klik på kryds i øverste højre hjørne forsvinder "Søger - vent venligst..."

Reserveringen er gennemført, så det er ikke dét, der er problemet.

Problemet er registreret både på vanilla og upgrade.

History

#1 Updated by Simon Holt over 1 year ago

Hej Tue,

Kan du prøve at se om der kommer en fejl i console i browseren?

Ligeledes i den anden sag.

#2 Updated by Tue Gaston over 1 year ago

Simon Holt wrote:

Hej Tue,

Kan du prøve at se om der kommer en fejl i console i browseren?

Ligeledes i den anden sag.

Det vil jeg vældig gerne, hvis du kan fortælle mig, hvordan jeg gør?

#3 Updated by Tue Gaston over 1 year ago

Tue Gaston wrote:

Simon Holt wrote:

Hej Tue,

Kan du prøve at se om der kommer en fejl i console i browseren?

Ligeledes i den anden sag.

Det vil jeg vældig gerne, hvis du kan fortælle mig, hvordan jeg gør?

Okay, det lykkedes mig at google mig til det, og den smider flg.:

#4 Updated by Simon Holt over 1 year ago

Tak for det :)

Ser ud som om det "bare" er den warning, som også er omtalt i https://platform.dandigbib.org/issues/3199. Altså den burde jo egentlig at være fikset ifb. med opdatering af jQuery(?), men tror ikke det er den, der driller her. Vi har jo fint kunnet resevere med den warning før.

Så det må være noget andet, men det var forsøget værd lige at tjekke.

#5 Updated by Tue Gaston over 1 year ago

  • Description updated (diff)

#7 Updated by Gitte Barlach about 1 year ago

  • Status changed from New to Needs code review
  • Assignee changed from Christel Krabbenhøft to Kasper Garnæs
  • Target version set to Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0)

#8 Updated by Gitte Barlach about 1 year ago

  • Priority changed from Normal to Urgent

#9 Updated by Kasper Garnæs about 1 year ago

For the record så er denne forårsaget af jQuery opdateringen i #3603.

#10 Updated by Jesper Kristensen about 1 year ago

Hej Simon

Jeg er ikke sikker på jeg helt forstår dette fix. Vil det ikke gøre at søge overlay ikke vise på colleciton og object?

F.eks. hvis man trykker på et emne ord inder https://www.aakb.dk/ting/object/870970-basis%3A54472250 så skal den vise osse… den skal jo ikke kun vises på sider under /ting/search

#11 Updated by Simon Holt about 1 year ago

Hej Jesper.

Må indrømme jeg ikke har testet dette specifikt, men jeg tænker da selectoren ".ting-object a[href^="/search/ting/" vil fange disse emneordssøgninger også?

#12 Updated by Kasper Garnæs about 1 year ago

F.eks. hvis man trykker på et emne ord inder https://www.aakb.dk/ting/object/870970-basis%3A54472250 så skal den vise osse…

Jeg har testet at click på emneord under en materialesamling og på et enkeltstående materiale også viser overlayet.

Selectoren ".ting-object a[href^="/search/ting/" rammer også disse situationer.

#13 Updated by Kasper Garnæs about 1 year ago

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

Reviewed. Koden fejler et enkelt af vores checks. Følg gerne op på det.

#14 Updated by Simon Holt about 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

PR er opdateret og passer nu alle test :)

#15 Updated by Gitte Barlach about 1 year ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#16 Updated by Kasper Garnæs about 1 year ago

  • Status changed from Needs code review to Reviewed

Godkendt.

#17 Updated by Kasper Garnæs about 1 year ago

  • Status changed from Reviewed to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Merged.

#18 Updated by Nhi A Sy about 1 year ago

  • Status changed from Technical test to Resolved

#19 Updated by Nhi A Sy about 1 year ago

  • Status changed from Resolved to Technical test

#20 Updated by Nhi A Sy about 1 year ago

  • Assignee changed from Gitte Barlach to Nhi A Sy

#21 Updated by Nhi A Sy about 1 year ago

  • Status changed from Technical test to Resolved

Testet på release 7.x-4.6.0-rc6 på upgrade-fbs.ddbcms.dk

Testet med brugeren Caroline Conrad DDB

Modal pop, vent venligst, forsvinder rimelig hurtig, hænger ikke de 15 min. mere.

Ingen warning i console

 

 

#22 Updated by Gitte Barlach about 1 year ago

  • Status changed from Resolved to Resolved (tag version)

Also available in: Atom PDF