search

stephenhmarsh / old_12parsecs

Old repo, I now live at "twelveparsecs"
https://github.com/stephenhmarsh/twelveparsecs
0 stars 0 forks source link

Parsecs aren't related to media (via an AR relationship)? #1

Open adambray opened 10 years ago

adambray commented 10 years ago

Hey @stephenhmarsh,

This is by no means necessary, but I wondered if you had a reason for only including the media title in a Parsec, as opposed to an ActiveRecord relationship?

In some ways, your ERD is already pretty complex, so I don't want to suggest that you make it more so, but maybe something to consider.

-Adam

adambray commented 10 years ago

(I got this info from looking at your ERD)

stephenhmarsh commented 10 years ago

Sorry man I made like 4 more ERD's after that first draft and didn't upload any of them. Got a new, saner ERD uploaded now: https://github.com/stephenhmarsh/12parsecs

I am just doing a catch-all "Media" table now.

On Fri, Mar 28, 2014 at 9:47 AM, Adam Bray notifications@github.com wrote:

Hey @stephenhmarsh https://github.com/stephenhmarsh,

This is by no means necessary, but I wondered if you had a reason for only including the media title in a Parsec, as opposed to an ActiveRecord relationship?

In some ways, your ERD is already pretty complex, so I don't want to suggest that you make it more so, but maybe something to consider.

-Adam

Reply to this email directly or view it on GitHubhttps://github.com/stephenhmarsh/12parsecs/issues/1 .