Skip to content

Conversation

@alliomeria
Copy link
Contributor

For esmero/ami#185, to complement esmero/ami#186

Update logic for processing MeSH queries:

  • for count > 0 (instead of 1)
  • make use of array_unshift to better sort and select results that more closely match original query

Coded live by @DiegoPino and shared with @alliomeria for local testing. Thank you @DiegoPino!

For esmero/ami#185, to complement  esmero/ami#186

Update logic for processing MeSH queries:
- for count > 0 (instead of 1)
- make use of array_unshift to better sort and select results that more closely match original query

Coded live by @DiegoPino and shared with @alliomeria for local testing. Thank you @DiegoPino!
@alliomeria alliomeria self-assigned this Nov 28, 2023
@alliomeria alliomeria requested a review from DiegoPino November 28, 2023 19:45
@alliomeria alliomeria added the LoD and Auth Controller A Tree of Apples label Nov 28, 2023
@alliomeria alliomeria changed the base branch from main to 1.3.0 November 28, 2023 19:46
@alliomeria alliomeria added this to the 1.3.0 milestone Nov 28, 2023
'label' => $entry['label'],
];
}
$input_encoded = rawurlencode($input);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alliomeria bit like the last one, could you in this case restore the indentation? Basically move everything right ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How's this now @DiegoPino ?

Copy link
Member

@DiegoPino DiegoPino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing! thanks so much

@DiegoPino DiegoPino merged commit 1ccf67b into 1.3.0 Nov 29, 2023
@DiegoPino DiegoPino deleted the alliomeria-patch-1 branch November 29, 2023 17:33
@alliomeria
Copy link
Contributor Author

Thank you for your review & feedback @DiegoPino !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LoD and Auth Controller A Tree of Apples

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants