Skip to content

Conversation

vishv843
Copy link
Contributor

image
These are some of the stats I have added for now

@vishv843 vishv843 requested a review from a team as a code owner August 29, 2024 16:18
@vishv843 vishv843 force-pushed the feat/help-thread-stats branch 3 times, most recently from 9817bac to 9dfda29 Compare August 29, 2024 19:22
@vishv843 vishv843 force-pushed the feat/help-thread-stats branch from 9dfda29 to 8e1ae7d Compare August 29, 2024 19:29
@ankitsmt211
Copy link
Member

@vishv843 there's been a some sort of confusion, just going to rewrite the comment for visibility. The goal was to use database table help_threads to showcase these stats. That was the main reason we started storing some extra data per thread, if you look at the description for this issue carefully
Screenshot 2024-08-31 at 5 48 18 PM

these are the fields that are now stored as well per thread and can be utilized to showcase stats for help-threads.

However it seems you're pulling data from discord itself, which is not something we want.

Copy link
Member

@Zabuzard Zabuzard left a comment

Choose a reason for hiding this comment

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

as anki explained, the data should be stored and gathered from the DB in constant time instead of querying discords api for the stats.
sorry for the confusion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants