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

Notifikace pro stav DX #435

Closed
CIZKOVAL opened this issue Jul 1, 2022 · 8 comments
Closed

Notifikace pro stav DX #435

CIZKOVAL opened this issue Jul 1, 2022 · 8 comments
Assignees
Labels

Comments

@CIZKOVAL
Copy link
Collaborator

CIZKOVAL commented Jul 1, 2022

Dobrý den,
záznamy, které se dostanou do kurátorského stavu DX (typicky je důvodem smazání v SKC a následný update), tak chodí v notifikacích s posledním stavem, který mají, a který se nezměnil, protože došlo k update a tím pádem ke změně kurátorského stavu.

mapy_stav

historie_stavu_mapy

historie_mapy_kuratorsky_stav

NOT_mail

@CIZKOVAL CIZKOVAL added bug Something isn't working p:TEST Prostredi - Test labels Jul 1, 2022
@CIZKOVAL
Copy link
Collaborator Author

CIZKOVAL commented Jul 1, 2022

Nabízí se řešení, že by při změně stavu na DX nechodily nikomu notifikace, stejně jako u stavu D, viz issue #433 .

Nechám na zvážení p. @Lichtenbergova, @Cypzirschova

@Cypzirschova
Copy link
Collaborator

Souhlasím s paní @CIZKOVAL - při změně na DX by neměla chodit notifikace, protože se v notifikacích vlastně opakuje stejný veřejný stav (A, PA, NL).

@Cypzirschova
Copy link
Collaborator

Doplňuji, že notifikace chodí i u kurátorského stavu PX. Je potřeba, aby jakákoliv změna/doplnění kurátorského stavu nebyla důvod k tomu, aby chodily notifikace, když se veřejný stav NEZMĚNIL. Kurátorské stavy jsou pro kurátory a jejich práci s díly.

@Cypzirschova
Copy link
Collaborator

Cypzirschova commented Jan 10, 2024

U stavu DX situace stále stejná ve verzi 1.0.9.4. Je potřeba to změnit, protože některé knihovny mají nastavené notifikace pro svá díla.

Kurátorské stavy DX, PX, NPA - nemají chodit žádné notifikace.

@CIZKOVAL
Copy link
Collaborator Author

Retest dopadl následovně:
DX - při změně na kurátorský stav DX NEPŘIJDE notifikace = OK
D - při změně na kurátorský stav D NEPŘIJDE notifikace = OK
NPA - ještě čekáme na výsledky, předpokládáme, že bude OK, když se nemění běžný stav (pouze kurátorský)
PX - při změně na kurátorský stav PX PŘIJDE notifikace = NOK

historie_PX
notifikace_PX

@Cypzirschova
Copy link
Collaborator

Cypzirschova commented Feb 29, 2024

NPA je OK - notifikace nepřišla

Bug je tedy jen u PX.
@pavel-stastny

pavel-stastny added a commit that referenced this issue Mar 5, 2024
@Cypzirschova
Copy link
Collaborator

Cypzirschova commented Mar 8, 2024

Retest - 1.0.9.8 - DEV:
PX:
Stále chodí notifikace pro PX (oai:aleph-nkp.cz:SKC01-000832229):

image
E-mail:
image
Nesouvisí to s propsáním do klasického stavu - historie stavu?
@pavel-stastny

@Cypzirschova
Copy link
Collaborator

Cypzirschova commented Apr 3, 2024

Retest verze 1.1.0.0. - prostředí test:
PX - notifikace již nechodí - OK

@CIZKOVAL prosím uzavřít.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants