Daily Reset for the Token System

JeremyMerkur / Markus

Git repository of MarkUs

http://www.markusproject.org

MIT License

0 stars 0 forks source link

Daily Reset for the Token System #9

Closed nmaraston closed 11 years ago

nmaraston commented 11 years ago

Implement the ATE tokens system as defined in the specification.

We need to support tokens allocation in three different ways:

A static amount of tokens
Tokens that get reset daily
Unlimited tokens

Ensure to develop modularly so we may easily update later, keeping in mind expansion of adding by hour, week.

Investigate what is currently supported and refer to the specification for more UI details.

mpalermo14 commented 11 years ago

As we discussed yesterday, it does look like the daily and unlimited tokening system is implemented but we will need to review this as it seems more complicated than it needs to be.

nmaraston commented 11 years ago

As stated, the daily reset on tokens was indeed already implemented. However, the tokens are only allocated to students when a valid group is made. This assumes that the instructor setup the ATE before publishing the assignment. This is a risky assumption. If the instructor configures the ATE after groups are made, those groups do not receive tokens.

I have two possible solutions:

Implement a button that allows the instructor to reset all tokens allocated to groups.
Detect when the number of daily tokens for the assignment has changed and adjust each groupings tokens appropriately.

nmaraston commented 11 years ago

I made a separate issue for this: https://github.com/JeremyMerkur/Markus/issues/15

nmaraston commented 11 years ago

Can we close this? The token system has already been implemented to spec.

mpalermo14 commented 11 years ago

yeah I think it is safe to close

On Wed, Oct 30, 2013 at 11:50 PM, Nick Maraston notifications@github.comwrote:

Can we close this ticket? The token system has already been implemented to spec.

— Reply to this email directly or view it on GitHubhttps://github.com/JeremyMerkur/Markus/issues/9#issuecomment-27459391 .

nmaraston commented 11 years ago

Im closing this.

mpalermo14 commented 11 years ago

Sounds good.

On Wed, Nov 6, 2013 at 2:41 PM, Nick Maraston notifications@github.comwrote:

Closed #9 https://github.com/JeremyMerkur/Markus/issues/9.

— Reply to this email directly or view it on GitHubhttps://github.com/JeremyMerkur/Markus/issues/9 .