The whitespacing here is a little wonky.  The blank line on line 21 can be removed, and the id = on line 28 is indented one too many.

I'm also wondering if the id should be a Symbol.  I'm not a big fan of returning String objects when there are specific values.