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

Listing endorsements #3062

Closed
benhalpern opened this issue Jun 6, 2019 · 3 comments
Closed

Listing endorsements #3062

benhalpern opened this issue Jun 6, 2019 · 3 comments

Comments

@benhalpern
Copy link
Contributor

benhalpern commented Jun 6, 2019

I threw this idea out in our Slack a few weeks ago and @jessleenyc liked it. I still think it's a good idea and we should pursue it.

Listing endorsments

sloppy mockups:

endorsement-list-view
endorsement-show-view

Similar to a comment, but shorter and not necessarily part of a conversation. More a quick word from the community about the listing in mind. It allows listings to accrue social validation so that it makes sense to "bump" a listing as opposed to making a new one. Right now that benefit and flow is less clear.

All in all I think it adds a warm community feel, and all endorsements have a profile attached that can be clicked through to.

LinkedIn endorsements have created a somewhat cringey association, but I think this is a much more natural place to endorse a specific situation, rather than a "human". I think the endorsements can add some details and second opinions that could really add to a listing.

I think the lister should have the right to choose which endorsements get displayed. This is not a comment for general discussion, it is specifically to reinforce the value or ideas of the listing.

We should also establish a protocol for negative ratings, etc. in the future but I don't think that is such a straightforward thing to do. Any attempts to introduce a truly effective rating system or anything like that is bound to be gamed, so I think starting with a simple rating system while continuing to be diligent about responding to the report abuse button and working with the community to make sure listings are honest and helpful.

Technical details.

A ClassifiedListing object would have a has_many relationship with its endorsements. The endorsements would have a user_id and classified_listing_id and all the things you'd expect. This should all be pretty straightforward.

The endorsement info will have to be stored in Elasticsearch index likely with the classified listing.

Endorsements should have an approved or displayed boolean or something like that for the author of the listing to switch on. They should probably be able to manage it through their listings dashboard and also get a notification when a new endorsement comes in, which they can act on.

Please let me know if you intend to work on this!

@neilgreen
Copy link

@benhalpern I know you mentioned that someone should let you know if they intend to work on this feature but I didn't want to commit to doing it if I wasn't sure I could get it done, and the only way I knew to be sure I could get it done was to do it. If the feature doesn't work the way you expect, or it's not implemented correctly let me know and I'll adjust it accordingly.

@Oghenebrume50
Copy link
Contributor

A ClassifiedListing object would have a has_many relationship with its endorsements. The endorsements would have a user_id and classified_listing_id and all the things you'd expect. This should all be pretty straightforward.

Hi @benhalpern I would like to work on this, going by this statement, I cannot find the ClassifiedListing model in the model folder, however, I see the listing model, is this what you were referring to here?

@rhymes
Copy link
Contributor

rhymes commented Jul 2, 2020

Hi @Oghenebrume50! It was recently renamed to just Listing. They are the same thing :) I'm going to assign you to the feature

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

No branches or pull requests

8 participants