Skip to content

Fix: Incorrect access count in topics EE#1038

Merged
mohdsayed merged 1 commit into
release/v0.14.1from
fix/topics-access-count
Apr 10, 2025
Merged

Fix: Incorrect access count in topics EE#1038
mohdsayed merged 1 commit into
release/v0.14.1from
fix/topics-access-count

Conversation

@mayan-000

@mayan-000 mayan-000 commented Apr 9, 2025

Copy link
Copy Markdown
Collaborator

Description

This PR fixes incorrect access counts by resetting the topics table data for an epoch if it was not completed in the first render.

Relevant Technical Choices

Testing Instructions

  • Open PSAT and navigate to topics EE.
  • Let the animation run for a bit, then click on any topic in the table, and this should open the tax tree.
  • Now come back to the EE tab, the access count should be correct.
  • Now resize the table tray, and the data should remain correct.

Additional Information:

Screenshot/Screencast


Checklist

  • I have thoroughly tested this code to the best of my abilities.
  • I have reviewed the code myself before requesting a review.
  • This code is covered by unit tests to verify that it works as intended.
  • The QA of this PR is done by a member of the QA team (to be checked by QA).

Fixes #

@mayan-000 mayan-000 marked this pull request as ready for review April 10, 2025 07:46
@mohdsayed mohdsayed merged commit 7172774 into release/v0.14.1 Apr 10, 2025
@mohdsayed mohdsayed added this to the v0.14.1 milestone Apr 10, 2025
@mohdsayed mohdsayed deleted the fix/topics-access-count branch April 10, 2025 12:45
@mohdsayed mohdsayed mentioned this pull request Apr 21, 2025
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