cl0ne
/
cryptopotato-bot
1
0
Fork 0
Code Issues 9 Pull Requests Releases Activity
Labels Milestones
New Issue

Build titles message from latest given titles records #33

Open
opened 5 years ago by cl0ne · 0 comments
cl0ne commented 5 years ago
Owner

Currently latest composed titles texts are stored as Markdown-formatted fields within the GroupChat model. That was okay before the given titles records were implemented. Now it's both a rigid design approach and a redundant data in the database. Both fields are better to be replaced with single nullable integer last_given_titles_count field where value of NULL means no participants were found and 0 stands for empty title pools.

  • Rewrite assigned titles texts creation to use given title records
  • Add migrations
  • Cache formatted texts?
  • Test it
  • Polish
Currently latest composed titles texts are stored as Markdown-formatted fields within the `GroupChat` model. That was okay before the given titles records were implemented. Now it's both a rigid design approach and a redundant data in the database. Both fields are better to be replaced with single nullable integer `last_given_titles_count` field where value of `NULL` means no participants were found and `0` stands for empty title pools. - [x] Rewrite assigned titles texts creation to use given title records - [x] Add migrations - [ ] Cache formatted texts? - [x] Test it - [ ] Polish
cl0ne added the
Improvement
 label 5 years ago
cl0ne self-assigned this 5 years ago
Sign in to join this conversation.
feature/build-titles-from-stats
Branches Tags
Clear current reference
archive/feature/daily-titles
archive/feature/daily-titles-log
feature-flip-a-coin
feature-fuel-prices
feature/build-titles-from-stats
maintenance/update-dependencies
master
Labels
Apply labels
Clear labels
Bug CI/CD
CI/CD related stuff daily titles dice roll Feature request Improvement Maintenance Needs investigation Unconfirmed
No Label Bug CI/CD daily titles dice roll Feature request Improvement Maintenance Needs investigation Unconfirmed
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
cl0ne
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(MISSING)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes