Add MediaMessage result class issue #152

interchange / interchange6-schema

DBIC schema for Interchange 6

8 stars 7 forks source link

Add MediaMessage result class issue #152 #153

Closed hexfusion closed 9 years ago

hexfusion commented 9 years ago

add MediaMessage result class.
add Result::MediaMessage to MANIFEST.
add man_to_many relationship with to Message class
add might_have relationship with Media class.

NOTE: Media result class has been updated per #134 which is also done in PR #151. So merging 151 before this would probably make sense?

SysPete commented 9 years ago

Is it possible to have some fixtures and tests for this?

hexfusion commented 9 years ago

@SysPete sounds good will add this now.

SysPete commented 9 years ago

@sbatschelet on the other hand isn't it easier just to use markdown/whatever in message content with img tag then we don't need anything extra in db?

hexfusion commented 9 years ago

@SysPete It would be nice to have the ability to manage the Media assets used by Message esp for anonymous user. So for example if an anonymous user wants to add a picture to a review. Markdown is certainly doable but then you lose some control is all. Or if user is role => 'editor' he can have access to product media in blog. I hear ya about keeping it simple.

racke commented 9 years ago

In my opinion we should make sure first that the structure of the existing media tables is sound.

hexfusion commented 9 years ago

@racke does anything stick out to you re: Media?