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.

Sign up for GitHub

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

Jump to bottom

Merge to master: Release 2.x #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Merge to master: Release 2.x #26

wants to merge 2 commits into from

Conversation

syphax-bouazzouni
Copy link

@syphax-bouazzouni syphax-bouazzouni commented Feb 3, 2025 •
edited
Loading

syphax-bouazzouni and others added 2 commits February 2, 2025 13:56
@syphax-bouazzouni
Feature: Add class level explorer to avoid unnecessary calls (#25)
23ae179 
* add class level explorer to avoid unnecessary calls

* add link explorer tests

* add link explorer forwarding

* optimize the top level links call to be done only once

* fix test after making top_level_links class variable
@Bilelkihal
Fix: federation requests timeout (#24)
f5b82f3 
* store portal status in the cache down, and prevent requests for dead portals

* set timeout for requests to 20s

* change timeout to 30s

* refactor request federation module

* add rails cache to tests envirenement

* adjust federation test to handle when a portal is not responding

* update cache using to not add rails gem

* update gemfile.lock

* initialize logger and update activesupport version

* fix rails cache usage in request federation to use UI cache, while keeping tests running

* remove unnecessary activesupport from gemfile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@syphax-bouazzouni @Bilelkihal