Remove Context from result #208

Merged
miriam.baglioni merged 3 commits from cleancontext into beta 2 years ago
Collaborator

first naive implementation of a functionality to clean not wanted contextes from one result. This implementation simply verifies the main title of the results start with a given string. If it is the case, the specified context is removed from those of the result if present

first naive implementation of a functionality to clean not wanted contextes from one result. This implementation simply verifies the main title of the results start with a given string. If it is the case, the specified context is removed from those of the result if present
miriam.baglioni added the
enhancement
label 2 years ago
alessia.bardi was assigned by miriam.baglioni 2 years ago
claudio.atzori was assigned by miriam.baglioni 2 years ago
miriam.baglioni added 1 commit 2 years ago
alessia.bardi requested changes 2 years ago
alessia.bardi left a comment
Owner

With this implementation we'll remove a given context if the main title (and only the main title) of the product starts (and only starts) with a given string.

I think this is what we wanted, just update the description of the parameters, the code looks ok to me.

With this implementation we'll remove a given context if the main title (and only the main title) of the product starts (and only starts) with a given string. I think this is what we wanted, just update the description of the parameters, the code looks ok to me.
@ -0,0 +20,4 @@
{
"paramName": "ci",
"paramLongName": "contextId",
"paramDescription": "url to the ISLookup Service",
Owner

Update param description

Update param description
miriam.baglioni marked this conversation as resolved
@ -0,0 +31,4 @@
},{
"paramName": "vf",
"paramLongName": "verifyParam",
"paramDescription": "url to the ISLookup Service",
Owner

Update param description

Update param description
miriam.baglioni marked this conversation as resolved
miriam.baglioni force-pushed cleancontext from 07d58568d2 to 79336d46c5 2 years ago
miriam.baglioni added 1 commit 2 years ago
Poster
Collaborator

Modified the descriptions of the parameters

Modified the descriptions of the parameters
miriam.baglioni added 1 commit 2 years ago
miriam.baglioni merged commit 61c0266a44 into beta 2 years ago
miriam.baglioni deleted branch cleancontext 2 years ago

Reviewers

alessia.bardi requested changes 2 years ago
The pull request has been merged as 61c0266a44.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b cleancontext beta
git pull origin cleancontext

Step 2:

Merge the changes and update on Gitea.
git checkout beta
git merge --no-ff cleancontext
git push origin beta
Sign in to join this conversation.
No reviewers
No Milestone
No project
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: D-Net/dnet-hadoop#208
Loading…
There is no content yet.