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

Add required columns to 'Manage Events/Sessions' for rating #3195

Closed
shreyanshdwivedi opened this issue Jun 25, 2019 · 3 comments · Fixed by #3198
Closed

Add required columns to 'Manage Events/Sessions' for rating #3195

shreyanshdwivedi opened this issue Jun 25, 2019 · 3 comments · Fixed by #3198

Comments

@shreyanshdwivedi
Copy link
Member

shreyanshdwivedi commented Jun 25, 2019

Sub-issue of #2959

We give any role that has the right to rate

  • the option to choose any star from 1 to 5

Also then we can have two more column showing

  • average rating
  • number of people who rated

Columns Required - | Rate session | Avg Rating | No. of people rated |

Additional Context
On it

@kushthedude
Copy link
Member

@shreyanshdwivedi I suggest you take this following feature on hold, Ember-Table refactor for Manage-Events will be ready in 2-3 days, I don't think we should at least, for now, make a change in manage-events model table, What you say ?

@shreyanshdwivedi
Copy link
Member Author

If @CosmicCoder96 feels the same, I can put a hold on it.

@abhinavk96
Copy link
Contributor

@shreyanshdwivedi If you are creating a separate cell component, then you can just go ahead, I am using the existing cell-components.

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