ElixirMostWanted / elixirmostwanted.com

58 stars 8 forks source link

Implement a data tables #3

Closed bcardarella closed 3 months ago

bcardarella commented 3 months ago

tables

wanteds:

comments:

Depends on #2

w0rd-driven commented 3 months ago

Would completed_at as a timestamp be more useful to capture when it was completed? Is this scoped to just the schema or would it be helpful to generate the views as well?

bcardarella commented 3 months ago

yes, the timestamps should be there. I omitted them as I always presume they're there. I'll add to the spec

bcardarella commented 3 months ago

@w0rd-driven oh sorry, I misunderstood. You want to change completed to a timestamp. That works for me. Updating

greven commented 3 months ago

Hi,

Created a PR to this issue: #8.

bcardarella commented 3 months ago

Closed by #8