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

FEATURE: Add per-reaction score awarding #95

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rngus2344
Copy link
Contributor

Now giving and receiving reactions award users per-reaction score:

Screenshot 2023-06-07 at 1 56 18 PM Screenshot 2023-06-07 at 1 55 09 PM Screenshot 2023-06-07 at 1 56 50 PM

@xfalcox
Copy link
Member

xfalcox commented Jun 7, 2023

AFAIK the default reaction also creates a like, so it's double counting both likes and reactions here? Not sure we can do something about it tho.

@rngus2344
Copy link
Contributor Author

rngus2344 commented Jun 7, 2023

AFAIK the default reaction also creates a like, so it's double counting both likes and reactions here? Not sure we can do something about it tho.

When I checked it, the default reaction was only counted as a like not as reaction. I will double-check it and let you know.

@rngus2344
Copy link
Contributor Author

rngus2344 commented Jun 7, 2023

When I do default reaction, it creates 'like' and is stored in post_actions:
Screenshot 2023-06-07 at 2 27 33 PM

But it is not stored in discourse_reactions_reaction_users:
Screenshot 2023-06-07 at 2 27 00 PM

(The post_id 87 is where I did 'like')
Is it correct? Please let me know if I am missing something.

@janzenisaac janzenisaac requested a review from xfalcox June 20, 2023 14:40
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