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

Feature: Align to Agroportal 3.0 #8

Conversation

syphax-bouazzouni
Copy link
Collaborator

No description provided.

syphax-bouazzouni and others added 24 commits October 16, 2023 14:38
* add tests for the api cache code

* refactor the cache middleware code

* remove unused function in the cache middleware
…e-apis

Feature: implement federation of collections calls
…e-apis

Fix: in parallel calls save main thread states a cross threads
* handle ssl error exception when federated call to a collection

* federate search endpoint and merge results

* fix activesupport gem version issue

* fix search params to keep federated and non federated calls work

* put collections mergin only for class search

* pass federation errors in federated search results

* remove unused parallel dependency in class.rb

* clean class.rb federation code

* update the activesupport gem version

* remove uncompleted federated search test

* use the key "collection" instead of "results" in federated search result hash to maintain the endpoint expected schema

* add test for federated search
#20)

* add rails performance gem

* save as a custom event the API calls

* disable by default the rail performance monitoring

* use directly RailsPerformance::Models::CustomRecord instead of measure
* return search results as an object instad of a hash

* update test federated search to use search results as an object instead of a hash

* fix tests timeout by using the dev server of biodivportal

---------

Co-authored-by: Syphax <[email protected]>
@syphax-bouazzouni syphax-bouazzouni merged commit 36cbe87 into lifewatch-eric:master Oct 29, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants