-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implement bulk actions #3
Comments
In GitLab by @vronk on Nov 3, 2021, 13:40 mentioned in issue sshoc-marketplace-backend#128 |
In GitLab by @vronk on Nov 3, 2021, 13:40 marked this issue as related to sshoc-marketplace-backend#128 |
In GitLab by @vronk on Nov 3, 2021, 13:43 We agree to try to implement this via notebooks first. |
In GitLab by @vronk on Feb 2, 2022, 15:12 Notebooks seem mostly a solution, we keep this open as low-priority if we see that there are operations which would need server-side processing. |
In GitLab by @laureD19 on Jun 22, 2022, 11:57 marked this issue as related to sshoc-marketplace#84 |
In GitLab by @KlausIllmayer on Sep 14, 2022, 10:43 The workflow would look like this:
|
In GitLab by @KlausIllmayer on Sep 14, 2022, 10:46 What we need to check: if the GET JSON response could be used 1:1 for the PUT endpoint JSON. We had issues in the past about this, they should be solved but I never checked it for all of the possible values. We could create on stage an item that has all values filled out and test it with this one, if after running the script it is 1:1 as the ingested version. |
In GitLab by @KlausIllmayer on Sep 14, 2022, 12:34 The API call to get all ingested items from one specific source: |
In GitLab by @laureD19 on Oct 26, 2022, 15:42 moved from sshoc-marketplace#89 |
We discussed, that we need not only a bulk approval but also a bulk reject workflow. After the ingestion pipeline (re-)harvested data from a source, these items get the status Here is an example workflow, with the option to approve or the option to reject the items, including the API endpoints to use:
@cesareconcordia I hope the workflow is now more clearer and I hopefully covered all necessary steps, if not, please comment in this issue. There are also examples on the stage and on the development instance of marketplace. Things to check together with @laureD19:
|
Adding a point which needs to be handled: if there are two re-ingests of the same source, we will have of one item two different versions. This can be seen in the table, as there will be two entries with the same persistentId and label. In such cases, we need an agreement how to handle such a situation. Going through the proposed workflow alone could lead to irritating results (depending on the sort algorithms it may approve a version of the first ingest or of the second ingest). Most probably, the script will also run in an error, as the approval of a version will reject all other versions, that are then not available anymore. |
@KlausIllmayer : workflow seems clear, thanks. Will talk about its implementation during the next EB call |
initial test for bulk rejection of ingested items available here could you have a look @cesareconcordia and @KlausIllmayer and tell me what should be improved? |
review and create smaller tasks |
In GitLab by @KlausIllmayer on Jul 1, 2021, 18:22
We need bulk actions when a lot of items are involved like approving an ingest. There are - as I see it - three ways how to handle this (and in principle, all of this three ways could operate in parallel):
ingested
from sourcex
(disadvantage: take it or leave it - granularity could be a problem)Open for discussion to find a solution: @vronk @laureD19 @tparkola @egray523 @vronk @stefanprobst @cesareconcordia
The text was updated successfully, but these errors were encountered: