-
Notifications
You must be signed in to change notification settings - Fork 3
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
Improvements #1
Improvements #1
Conversation
last_watched_submission_at: DateTime.now | ||
) | ||
|
||
render json: User.current_watching_for(submission_id, current_user.id).to_a.map(&:nickname) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lesniakania I think it's good to use pluck
here http://api.rubyonrails.org/classes/ActiveRecord/Calculations.html#method-i-pluck
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, you are right, changed.
@lesniakania please rebase and resolve conflicts |
Conflicts: app/controllers/api/submissions_controller.rb
Some refactors.
Added authorization.
Added automatically rejecting mechanism.
Little improvements of calculating avg rate.
Small changes in views.
@szemek would you have some time tomorrow to look at this pull request together?
@gitjul would you have some time to help me testing if I didn't break anything?