 <p>The only thing I personally would have done differently is:</p> <ul> <li>Use EF (but yeah, L2S is fine for a quickie like this)</li> <li>Create a ViewModel class for the details View. I use ViewData as sparingly as possible.</li> </ul> <p>But yeah, nothing bad here and I agree that it was probably not a good place to work at. People also tend to make up vague reasons if they didn't want to say they didn't like your face.</p> <p>/shrug</p>