- **status**: review --> closed
- **assigned_to**: Shalitha Suranga
- **Reviewer**: Dave Brondsema
- **Comment**:

Merged at https://forge-allura.apache.org/p/allura/git/merge-requests/308/



---

** [tickets:#8253] Adding reaction support for comments**

**Status:** closed
**Milestone:** unreleased
**Created:** Fri Oct 26, 2018 07:45 AM UTC by Shalitha Suranga
**Last Updated:** Thu Nov 08, 2018 06:08 AM UTC
**Owner:** Shalitha Suranga


This is related with merge request 
[#135](https://forge-allura.apache.org/p/allura/git/merge-requests/135/) 
comments. 

VotableArtifact stores +1 and -1 in seperate fields 
```
    votes_up = FieldProperty(int, if_missing=0)
    votes_down = FieldProperty(int, if_missing=0)
```

So can we create something like ReactableArtifact

```
    react_thumbs_up = FieldProperty(int, if_missing=0)
    react_thumbs_down = FieldProperty(int, if_missing=0)
    react_thumbs_tada = FieldProperty(int, if_missing=0)
    ....
    ..
    react_thumbs_up_users = FieldProperty(int, if_missing=0)
```

Another thing is that since Allura supports comment replies like a tree. we may 
ignore who reacted .. we just take count only. But normally it is needed


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed 
to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is 
a mailing list, you can unsubscribe from the mailing list.

Reply via email to