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

Ignore old responses for the DownloadsPage #7391

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

drew2a
Copy link
Contributor

@drew2a drew2a commented Apr 25, 2023

This PR is related to #7360 and continues the work done in #7378.

It adds an id for each Request on the GUI side. This new functionality is utilized in the DownloadsPage to ignore outdated requests.

@drew2a drew2a force-pushed the fix/7360_2 branch 2 times, most recently from aab3189 to 1d833bf Compare April 26, 2023 08:41
@@ -34,9 +34,10 @@ def __init__(self, limit: int = 50, timeout_interval: int = 15):
self.protocol = DEFAULT_API_PROTOCOL
self.host = DEFAULT_API_HOST
self.port = DEFAULT_API_PORT
self.key = b""
self.key = ''
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems like a bug that became visible after I wrote the tests. It's not directly related to the PR.

@drew2a drew2a force-pushed the fix/7360_2 branch 3 times, most recently from 732cb18 to b3dac11 Compare April 26, 2023 08:51
@drew2a drew2a requested a review from kozlovsky April 26, 2023 08:51
@drew2a drew2a marked this pull request as ready for review April 26, 2023 08:51
@drew2a drew2a requested a review from a team April 26, 2023 08:51
@drew2a drew2a merged commit 3a31c77 into Tribler:release/7.13 Apr 28, 2023
@drew2a drew2a deleted the fix/7360_2 branch April 28, 2023 09:08
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.

2 participants