Extend link functionality

djstevenson commented 4 years ago

I want

To be able to refer to a link in my Markdown and have it present something sensible
To have a list of links at the end which may, or may not, include the above
To have a list of 'other versions'.

We currently have id, song_id, name, url, description, priority, extras (which is for stuff like YouTube aspect ratio, and/or start time, etc).

"description" is text that would appear in the link, but we'll want to style different destinations differently (e.g. add a spotify logo?).

"name" doesn't have a strict-enough definition for me to do what I want.

djstevenson commented 4 years ago

Proposals:

Priority can be 0, which means don't show in link list at the end.
name is replaced with 'identifier' and 'class'.
- Identifier would be unique per song (UNIQUE constraint on song_id, identifier) and would be how we refer to a link in Markdown.
- Class will be "youtube", "version", etc. It might even be a drop down and become class_id. In which case we'll need crud for classes. We could derive a CSS class name from this, perhaps.

djstevenson commented 4 years ago

Fixed in PR #191

djstevenson / songs-to-the-siren