search

buerokratt / Maintenance

Repo with temporary content
MIT License
0 stars 0 forks source link

NLib debug #1

Closed turnerrainer closed 1 year ago

turnerrainer commented 1 year ago

AS National Library I WANT our GUI to be Bürokratt widget compatible SO THAT I could start using Bürokratt

Acceptance Criteria

turnerrainer commented 1 year ago

Prod without Bürokratt chat widget

https://dea.digar.ee/cgi-bin/dea?a=cl&cl=CL1&sp=arevalla

Prod with Bürokratt chat widget

https://testdea.nlib.ee/?a=cl&cl=CL1&sp=arevalla&e=-------et-25--1--txt-txIN%7ctxTI%7ctxAU%7ctxTA-------------

Kristjanmn commented 1 year ago

Leidsin vead kolmes funktsioonis.

  1. $("#editvaluepopup").on('hide.bs.modal', function (e) { return closeEditValuePopup(); });
  2. $(function() { if (cookie.get('deacookiepolicy') == undefined) { $("#deacookienotificationbarcontainer").show(); } });
  3. $(function() { $('#useraccountpageuserliststab').on('shown.bs.tab', function (e) { getUserLists(); }); });

Tundub et javascript ei tunne seal jQueryt ära. Üks võimalik lahendus on asendada vigastel funktsioonidel $ ära $->jQuery vastu. Kui see ei aita siis võib ka proovida ilma dollari märgita

(function() {
//....siia see kood mis pohjustas vea
})()

või siis natukene kindlam variant

document.addEventListener("DOMContentLoaded", () => {
  //....siia see kood mis pohjustas vea
});

Ma loodan et see aitab probleemi lahendada.

turnerrainer commented 1 year ago

Won't fix as business requirements changed.