add shrimpo models/tables #204
Conversation
app/models/experience_point_award.rb
Outdated
| @@ -15,6 +15,7 @@ class ExperiencePointAward < ApplicationRecord | |||
| :streamingatron, # streaming live | |||
| :glorppy, # daily glorp lottery | |||
| :gloppy, # daily glop lottery | |||
| :shrimpo_prize, # winning shrimpo | |||
There was a problem hiding this comment.
i guess it seems a little repetitive to say shrimpo_prize when its already a shrimp. There's only one winner per shrimpo right? maybe it can be :first_prize, :trophy_winner, or whatever??
There was a problem hiding this comment.
Well the thing is this is used directly in the notification text:
when "experience_point_award"
"You got #{self.source.amount} #{self.source.award_type} points!"`So I was wondering if "You got 500 shrimpo prize points!" would sound ok or if we should change it
There was a problem hiding this comment.
in that case maybe just 'shrimpo' is fine
There was a problem hiding this comment.
yeah that message would make the most sense for now 🦐
| after_create :queue_end_shrimpo_job | ||
|
|
||
| VALID_DURATIONS = [ | ||
| # minors |
There was a problem hiding this comment.
entries to a minor shrimpo are anonymized until the shrimpo is over
entries to a major shrimpo are public even before the voting starts
There was a problem hiding this comment.
oh wow ok didn't even think about that but it makes sense to me. how does one choose one type over the other?
There was a problem hiding this comment.
anything less than 1 month is minor otherwise major
| @@ -0,0 +1,2 @@ | |||
| class ShrimpoVotingCategory < ApplicationRecord | |||
There was a problem hiding this comment.
remove this or maybe we'll use it later IDK yet
| @@ -0,0 +1,12 @@ | |||
| class CreateShrimpoVotingCategories < ActiveRecord::Migration[7.0] | |||
| def change | |||
There was a problem hiding this comment.
remove this table or maybe we'll use later IDK yet
No description provided.