Skip to content

Severin Gehwolf jerboaa

  • Joined on Aug 07, 2010

Organizations

MarkUs
Severin Gehwolf
jerboaa merged pull request MarkUsProject/Markus#1327
Severin Gehwolf
Remove old fix in Rakefile
1 commit with 2 additions and 13 deletions
Severin Gehwolf
  • Severin Gehwolf e877b28
    Merge pull request #1330 from benjaminvialle/ECN-students
  • Benjamin Vialle 4eb2bee
    Merge branch 'master' of github.com:benjaminvialle/Markus into ECN-st…
  • 1 more commit »
jerboaa merged pull request MarkUsProject/Markus#1330
Severin Gehwolf
Ecn students
2 commits with 39 additions and 15 deletions
jerboaa commented on pull request MarkUsProject/Markus#1329
Severin Gehwolf

Patch seems OK to me. What I'm wondering about is if this require-all-defaults-to-be-defined is a good idea, though. I guess it would be less of an…

jerboaa commented on pull request MarkUsProject/Markus#1123
Severin Gehwolf

A functional test for this new action would be nice :)

jerboaa commented on pull request MarkUsProject/Markus#1121
Severin Gehwolf

Looks good, but it would be nice to have a comment about this issue in the code somewhere so as to avoid trying this in future again. Referencing t…

jerboaa commented on issue MarkUsProject/Markus#191
Severin Gehwolf

AFAIK, annotation categories are those "canned" annotations from which graders can select from. A limit of 255 chars seems fairly restrictive. Is i…

Severin Gehwolf
jerboaa merged pull request MarkUsProject/Markus#1112
Severin Gehwolf
Fix: array with ruby 1.9
1 commit with 1 addition and 1 deletion
Severin Gehwolf
  • Severin Gehwolf 1de4c27
    Merge pull request #1101 from benjaminvialle/update
  • Benjamin Vialle 06285d5
    Merge branch 'master' of github.com:benjaminvialle/Markus into update
  • 1 more commit »
jerboaa merged pull request MarkUsProject/Markus#1101
Severin Gehwolf
Fix: ruby 1.8 not set by default anymore
2 commits with 1 addition and 1 deletion
jerboaa commented on issue MarkUsProject/Markus#239
Severin Gehwolf

+1

jerboaa commented on issue MarkUsProject/Markus#1073
Severin Gehwolf

I see. Well if that's a use case you have, why not :)

jerboaa commented on issue MarkUsProject/Markus#1072
Severin Gehwolf

Could it be that you are just missing a db migration?

jerboaa commented on issue MarkUsProject/Markus#1073
Severin Gehwolf

Isn't this something the new assignments API would be able to do?https://github.com/MarkUsProject/Markus/blob/master/app/controllers/api/assignment…

Severin Gehwolf
jerboaa merged pull request MarkUsProject/Markus#1061
Severin Gehwolf
Added ECN students
1 commit with 4 additions and 0 deletions
Severin Gehwolf
jerboaa merged pull request MarkUsProject/Markus#1065
Severin Gehwolf
Fix: tests with remark request
1 commit with 23 additions and 21 deletions
jerboaa commented on pull request MarkUsProject/Markus#1037
Severin Gehwolf

Patch looks good. Fix the internationalization nits and it's good for merging.

jerboaa commented on pull request MarkUsProject/Markus#1037
Severin Gehwolf

Same here.

jerboaa commented on pull request MarkUsProject/Markus#1037
Severin Gehwolf

Please use I18n.t() for user visible strings. Thanks!

Severin Gehwolf
jerboaa merged pull request MarkUsProject/Markus#942
Severin Gehwolf
Fix Failing functional test
4 commits with 33 additions and 1 deletion
Severin Gehwolf
jerboaa merged pull request MarkUsProject/Markus#1012
Severin Gehwolf
Updated Rails version from 3.0.19 to 3.0.20
1 commit with 36 additions and 59 deletions
jerboaa commented on pull request MarkUsProject/Markus#1014
Severin Gehwolf

@danielstjules Please also update the wiki page as to how to use the new API if you get a chance. I expect this changed and we are lacking user doc…

jerboaa commented on issue MarkUsProject/Markus#1002
Severin Gehwolf

Yes, https://gist.github.com/benjaminvialle/4055313 is what I meant. It would be good to get that incorporated.

Something went wrong with that request. Please try again.