by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jun 22 2016 15:31
    michaelblyons opened #27
  • Apr 02 2016 22:34
    Skinner927 opened #26
  • Feb 26 2015 12:54
    CodeBrauer closed #24
  • Feb 26 2015 12:54
    CodeBrauer commented #24
  • Dec 05 2014 23:28
    walkerab opened #25
  • Nov 15 2014 22:48
    CodeBrauer opened #24
  • Nov 11 2014 21:35
    yaffeweb opened #23
  • Oct 31 2014 18:00
    halfer opened #22
  • Oct 30 2014 22:59
    xnite commented #21
  • Oct 30 2014 17:04
    ErikPeterson commented #21
  • Oct 29 2014 04:48
    xnite opened #21
  • May 19 2014 19:43
    eugmill commented #18
  • May 19 2014 19:40

    ErikPeterson on tidy-up-js

    (compare)

  • May 19 2014 19:40

    ErikPeterson on master

    Add comments. Switch .post to .… Fix typo. Add post attribute to query aja… and 1 more (compare)

  • May 19 2014 19:40
    ErikPeterson closed #20
  • May 19 2014 19:40
    ErikPeterson opened #20
  • May 19 2014 19:38

    ErikPeterson on tidy-up-js

    Add post attribute to query aja… (compare)

  • May 19 2014 19:34

    ErikPeterson on tidy-up-js

    Fix typo. (compare)

  • May 19 2014 19:33

    ErikPeterson on tidy-up-js

    Add comments. Switch .post to .… (compare)

  • May 19 2014 19:23

    ErikPeterson on add-reset-button

    (compare)

Erik Sälgström Peterson
@ErikPeterson
all it has in it is NaN
Eugene Millerman
@eugmill
so hot
i think my computer is sweating
Eugene Millerman
@eugmill
@ErikPeterson did you make the schema box bigger?
Eugene Millerman
@eugmill
is it alright if i change ID to id in the copy?
want to reserve caps for sql reserved words
Erik Sälgström Peterson
@ErikPeterson
Yeah go for it
Eugene Millerman
@eugmill
i also add missql : update_all
so minor changes to levels don’t have to wipe out the existing levels
Erik Sälgström Peterson
@ErikPeterson
nice
~(^_^)~
Sushanth Bhaskarabhatla
@bhaskaraspb
nice
Erik Sälgström Peterson
@ErikPeterson
Got has_col, has_row, is_record, and has_enough_rows test methods working.
i'm going to push branch but not merge
Eugene Millerman
@eugmill
nice, it’s sending error messages?
Erik Sälgström Peterson
@ErikPeterson
Ja, pull the branch and reload level one, I implemented tests for number of rows and correct column for that.
Eugene Millerman
@eugmill
niiiccee
Erik Sälgström Peterson
@ErikPeterson
\_(^_^)_/
Eugene Millerman
@eugmill
you sure you pushed?
Erik Sälgström Peterson
@ErikPeterson
lol markdown converted my emoji's hands to code block delimiters
yeah it's on the feature branch tho
read-tests-feature
git pull --all
lol whoops didn't commit anything to it one sec
ok pull now
Eugene Millerman
@eugmill
hmmm not seeing the errors
oh nevermind
it just needs more tests
wooot
"Try to imagine you're explaining SQL to Harry Truman” lol
want me to update the readme for the new level tests?
Erik Sälgström Peterson
@ErikPeterson
Yes plz
I'm going to add a few more test methods and try them out on the branch.
Eugene Millerman
@eugmill
sounds good
Eugene Millerman
@eugmill
why does a level have many user_databases again?
so the user_database knows which level is loaded?
Erik Sälgström Peterson
@ErikPeterson
yah
exactly
Eugene Millerman
@eugmill
can’t we just have a user_database belongs_to level?
we never really need level.user_databases ?
Erik Sälgström Peterson
@ErikPeterson
No we don't.
I think we wrote the other side of the association because it was breaking? But that might have been a different association
Maybe run the test suite w/o that side of the association and see
Eugene Millerman
@eugmill
it’s not a big deal ,just didn’t seem very semantic
Erik Sälgström Peterson
@ErikPeterson
I agree, we should remove it if we can.
I have working versions of: has_col, has_only_cols, has_only_col, has_row, has_only_row, is_record, has_enough_rows
Just pushed them up to the branch.
Sushanth Bhaskarabhatla
@bhaskaraspb