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

FIX: Store user_accuracy_bonus to clarify explanations #8165

Merged
merged 4 commits into from Oct 8, 2019

Conversation

@markvanlan
Copy link
Contributor

commented Oct 7, 2019

No description provided.

@markvanlan markvanlan requested a review from eviltrout Oct 7, 2019
@discoursebot

This comment has been minimized.

Copy link

commented Oct 7, 2019

You've signed the CLA, markvanlan. Thank you! This pull request is ready for review.

Copy link
Member

left a comment

Looks good so far!

@markvanlan markvanlan requested a review from eviltrout Oct 7, 2019
Copy link
Member

left a comment

I made a small comment about NULL but otherwise I think it's good to merge when ready!

@eviltrout eviltrout merged commit 057f698 into discourse:master Oct 8, 2019
1 check was pending
1 check was pending
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.