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
Switch to Rails 6.1 #365
Switch to Rails 6.1 #365
Conversation
@@ -31,7 +31,7 @@ class Mentor::ReplyToDiscussionTest < ActiveSupport::TestCase | |||
"foobar" | |||
) | |||
assert_equal 1, user.notifications.size | |||
notification = user.notifications.first | |||
notification = Notification.where(user: user).first |
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.
These are from me working on getting strict_loading working. It's problematic with FactoryBot so I've stopped for now. But these are still fine to merge.
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.
What's strict loading?
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.
It basically causes an exception if an n+1
might happen.
@@ -5,9 +5,10 @@ class Create | |||
initialize_with :solution, :submission | |||
|
|||
def call | |||
time = Time.now.utc.to_s(:db) |
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.
Nice.
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.
NOW()
doesn't work with the latest version of MySQL on mac for some reason.
No description provided.