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

fix: Fix timeout when checking if news are available #707

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

marienfressinaud
Copy link
Member

@marienfressinaud marienfressinaud commented Oct 3, 2024

Pull request checklist:

  • code is manually tested
  • interface works on both mobiles and big screens
  • accessibility has been tested
  • tests are updated
  • French locale is synchronized
  • documentation is updated (including comments, commit messages, migration notes, …)

If you think one of the item isn’t applicable to the PR, please check it
anyway and precise N/A next to it.

For some reasons, PostgreSQL doesn't use the same strategy to pick one
or more links in the NewsPicker. In production, picking one link can
take up to ~80 seconds with my account, while it takes less than 1s for
more links.
@marienfressinaud marienfressinaud merged commit 4cec5ba into main Oct 3, 2024
4 checks passed
@marienfressinaud marienfressinaud deleted the fix/news-available-too-long branch October 3, 2024 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant