FAIRsharing / fairsharing.github.io

The website for FAIRsharing, currently running at https://fairsharing.org.
https://fairsharing.org/
GNU Affero General Public License v3.0
8 stars 4 forks source link

Possibility to select multiple records to approve #2321

Open delphinedauga opened 4 months ago

delphinedauga commented 4 months ago

Approving a record in the curation dashboard often takes a few minutes (sometimes it's a second, sometimes, it's one ore two minute). Therefore, when we have to approve a dozen or so records (which happens frequently), it can take a long time. @allysonlister and I have observed the same behavior. Would it be possible to select several records that can be approved in one go? Or would it be possible to look at why approving a record takes so long and whether it can be optimised? Thanks !

knirirr commented 4 months ago

Have you noticed this problem with any particular registry or type of record? I ought to look into the speed first and so will be looking to try to reproduce it. If I can't I might need to ask one of you to share your screen with the developer tools open.

allysonlister commented 4 months ago

I'll check the timings, probably tomorrow when I do some curation, and get back to you. Happy to do a developer tools session if required, would be happy to just have a solution to the underlying problem, indeed.

knirirr commented 4 months ago

I've had a quick look at this myself and can't reproduce any delays; everything is quick on each approval. All that happens is a PUT in which data like these are sent:

{"fairsharing_record":{"approved":true,"skip_approval":true,"processing_notes":null,"create_review":true,"hidden":true}}

So, I think I'm definitely going to need to see it in action.