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

Implement API Rating field and widget #519

Closed
bajiat opened this issue Oct 8, 2015 · 16 comments
Closed

Implement API Rating field and widget #519

bajiat opened this issue Oct 8, 2015 · 16 comments
Assignees
Milestone

Comments

@bajiat
Copy link
Contributor

bajiat commented Oct 8, 2015

Feature

Create a rating feature for API Backends. Users can select a rating, and aggregate ratings are displayed on the API Backend page and API Backends catalog.

Design

Rating design

Resources

Related

Related to #471

@bajiat bajiat added the MVP2 label Oct 8, 2015
@brylie brylie self-assigned this Oct 26, 2015
@brylie brylie added this to the Sprint 15 milestone Oct 26, 2015
@brylie
Copy link
Contributor

brylie commented Oct 26, 2015

Co-pilot: @elnzv

@brylie brylie added ready and removed planning labels Oct 26, 2015
@55
Copy link
Contributor

55 commented Oct 26, 2015

@brylie brylie added in progress and removed ready labels Oct 27, 2015
@kyyberi
Copy link

kyyberi commented Oct 28, 2015

Somehow add the amount of ratings to option one. Could it be number in parenthesis after the stars? This is vital since its different level of credibility if there is just two ratings or 200 ratings.

@brylie
Copy link
Contributor

brylie commented Oct 28, 2015

This current feature development branch is focused on the basic, single-user rating widget. I have created a follow up design task to show the aggregate ratings (including number of ratings and average result).

@kyyberi
Copy link

kyyberi commented Oct 28, 2015

Ah, ok.

@brylie
Copy link
Contributor

brylie commented Oct 30, 2015

I am able to get the individual rating in a column of the API Catalog. Also, users who are anonymous or who have not rated the API Backend will see an average of existing ratings. There is an open bug report/feature request for search/sort within collection helper columns on the upstream package.

One problem I am getting is that we can not sort by the rating column, so users may not be able to 'show highest rated' by clicking the column title.

@brylie
Copy link
Contributor

brylie commented Oct 30, 2015

I have gone with a four star max rating, as opposed to five. This is to encourage users to lean one way or the other, rather than going with a three star rating.

@brylie
Copy link
Contributor

brylie commented Oct 30, 2015

I am not sure how to show both the aggregate rating and individual rating in the same table cell, but could create an additional column for 'average rating' or similar. This may not be the best in terms of space usage, but would differentiate the user rating from the aggregate.

@brylie
Copy link
Contributor

brylie commented Nov 2, 2015

I am implementing the rating widget in a similar fashion to Netflix movie rating functionality.

When a user has not rated an API backnd, they are shown the aggregate rating, e.g. red stars. When the user has rated an API backend, they are shown their individual rating, e.g. in yellow stars to differentiate.

@bajiat
Copy link
Contributor Author

bajiat commented Nov 2, 2015

@brylie I like the approach on separating the user ratings and aggregated ratings with different color.

@brylie
Copy link
Contributor

brylie commented Nov 2, 2015

PR Open: #572

@kyyberi
Copy link

kyyberi commented Nov 2, 2015

+1 for the idea

@brylie
Copy link
Contributor

brylie commented Nov 3, 2015

Below is a screenshot of the current API Catalogue table, showing user rating in yellow and aggregate rating in red:

screenshot

Note: the interface shows user rating when available, and falls back to aggregate rating or empty stars when no aggregate exists.

@brylie
Copy link
Contributor

brylie commented Nov 3, 2015

I have added the user count to aggregate rating display:
screenshot-1

A grey 'badge' appears next to the rating showing the aggregate count and a 'user' icon. The use of the icon avoids string translation and saves space.

@kyyberi
Copy link

kyyberi commented Nov 7, 2015

@brylie
Copy link
Contributor

brylie commented Nov 9, 2015

A lot of the ideas in that article seem also related to our API Metadata entry form task (#472). E.g. we can build an 'API Metadata' form that conforms to existing standards (including possibly Swagger specification), and provides fields such as outlined in the previous article.

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

No branches or pull requests

5 participants