Skip to content

fix: inconsistent loading of discussion thread response - #38

Merged
naincy128 merged 1 commit into
masterfrom
naincy/LP-529
Jul 6, 2026
Merged

naincy128 merged 1 commit into
masterfrom
naincy/LP-529

Conversation

@naincy128

@naincy128 naincy128 commented Jul 6, 2026 •

Copy link
Copy Markdown

Description

Fixes an issue where inline replies within discussion threads could load inconsistently, causing some replies to appear or disappear between page refreshes.

Root Cause

Response comments were not being retrieved using backend-level pagination. Combined with frontend request race conditions and stale state handling, this could result in inconsistent loading of inline replies, especially in threads with many responses.

As a result, users could see different combinations of replies on successive page refreshes, even though the underlying discussion data was unchanged.

Changes

Frontend (frontend-app-discussions)

  • Added request cancellation using AbortController to prevent stale requests from overwriting newer data.
  • Cleared stale child-comment state when reloading response comments.

Ticket

LP-529

@naincy128
naincy128 marked this pull request as ready for review July 6, 2026 10:14
Copilot AI review requested due to automatic review settings July 6, 2026 10:14
@naincy128 naincy128 closed this Jul 6, 2026

This comment was marked as resolved.

@naincy128 naincy128 reopened this Jul 6, 2026
Copilot AI review requested due to automatic review settings July 6, 2026 10:37

This comment was marked as resolved.

@naincy128
naincy128 merged commit acbad4d into master Jul 6, 2026
3 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.

3 participants