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

Previous in-app notifications about curation status get updated to latest curation status #10835

Open
lobbikerivm opened this issue Sep 11, 2024 · 8 comments
Labels
GDCC: QDR of interest to QDR Type: Bug a defect

Comments

@lobbikerivm
Copy link

lobbikerivm commented Sep 11, 2024

What steps does it take to reproduce the issue?
I start a curation process, and the status is changed multipletimes. I receive notifications in-app

  • When does this issue occur?
    after every new notification

  • Which page(s) does it occurs on?
    User> Notifications

  • What happens?
    Previous notifications get updated. The text of all the notification mentions the most recent curation status only

  • To whom does it occur (all users, curators, superusers)?
    superusers, didn't test other users

  • What did you expect to happen?
    Previous notifications should remain intact, so a user can see when the curation label is changed to what?

Which version of Dataverse are you using?
6.3

Any related open or closed issues to this bug report?
didn't see any

Screenshots:
curation_notification

Are you thinking about creating a pull request for this issue?
Help is always welcome, is this bug something you or your organization plan to fix?
I am not able to, not familiar with Java

@lobbikerivm lobbikerivm added the Type: Bug a defect label Sep 11, 2024
@pdurbin
Copy link
Member

pdurbin commented Sep 11, 2024

Interesting. As a sanity check are you talking about curation labels? This: https://guides.dataverse.org/en/6.3/admin/dataverses-datasets.html#configure-a-dataset-to-allow-use-of-a-curation-label-set

And when you say you're starting a curation process, can you please say a bit more about what you're doing?

@lobbikerivm
Copy link
Author

Yes, the are custom curation labels we use. I get a notification every time the curation label changes. Older in-app notifications for the same dataset get overwritten, so the all show the latest curation label (see the timestamps in the screenshot)

@lobbikerivm
Copy link
Author

curation_notification2

in these screenshots are three consecutive notification captures. Each taken after changing the curation label. You see the matching timestamps, with different contents of the notifications

@qqmyers
Copy link
Member

qqmyers commented Sep 11, 2024

FWIW: The existing design generates messages from the current db state, so it makes sense that all the curation state messages incorrectly show the current state. Since the initial development, an 'additionalInfo' field has been added to the UserNotification class - that could probably be used to store the at the time the notification was created.

@qqmyers qqmyers added the GDCC: QDR of interest to QDR label Sep 11, 2024
@lobbikerivm
Copy link
Author

I took a look in the database, and notice that the related thing stored in table usernotification is the objectid, which relates to the datasetversionid. In datasetversion is the current curation label stored. so the information about history of curation labels is not in the database. For a published dataset version the label is NULL, and the corresponding notification turns to 'The status of dataset sdsdf has been updated to .', as you would expect. Correct @qqmyers, the field additionalinfo in the database table is empty, and seems like a nice place to store the status at time of notification.

Thanks for having a look, guys.

Stefan

@pdurbin
Copy link
Member

pdurbin commented Sep 11, 2024

Are we thinking this is a new bug as of 6.3? Or does it go back several versions?

@qqmyers
Copy link
Member

qqmyers commented Sep 11, 2024

It's the way it has always worked/not worked. FWIW: The same thing is true for other messages, e.g. if you change the name of a dataset, the new name will show in all old messages about it, etc. Something like that may be OK, but its possible there are other messages that get confusing if you make changes later on.

@pdurbin
Copy link
Member

pdurbin commented Sep 11, 2024

Oooooh. I understand now. I agree this behavior is a bit odd. And yeah, I suppose the solution is to store more data in the database. I haven't looked at additionalInfo but, sure, it's probably worth a shot to store it there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GDCC: QDR of interest to QDR Type: Bug a defect
Projects
None yet
Development

No branches or pull requests

3 participants