-
Notifications
You must be signed in to change notification settings - Fork 121
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
Count points from optional pages #623
base: main
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #623 +/- ##
==========================================
+ Coverage 96.93% 96.94% +<.01%
==========================================
Files 45 45
Lines 11076 11079 +3
Branches 2060 2061 +1
==========================================
+ Hits 10737 10740 +3
Misses 292 292
Partials 47 47
Continue to review full report at Codecov.
|
@dzhuang ping :) |
See discussion in #621. I think what you want is a bonus page? |
Yeah, pretty much. Do we have a dfiferent mechanism for those? Because optional + has value makes a lot of sense to me to express that. |
I agree. So there are some agreements need to be reached to achieve that:
|
Only if they carry points.
Could simply be shown as "5 points (optional)". We may want a separate grade sum for "bonus points" on the results page--maybe. |
That sounds reasonable, but I have no time to do that in a while. Maybe I can help in testing if you can submit a patch. |
The earliest time where I envision I'll have some time to hack Relate is around Christmas. I'm teaching a big, Relate-heavy course in the spring, so I have some motivation. |
So I'll wait for your ping :) |
Closes #621
@dzhuang Could you review/comment?