Groove-Theory / Groove-Theory/Groovebot

Revisit Options.CheckServerOptionsExist, uncouple it's method

Open
#30 0 comments 0 reactions 0 assignees View on GitHub
enhancement requirements needed
Dominant language
JavaScript
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

Options.CheckServerOptionsExist seems awkward in it's implementation right now and where we use it, we should revisit this so it doesn't seem like spaghetti code as we check and/or insert option collection records for a guild.

Maybe find a way to auto-populate the table when a bot is invited as well?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating Options.CheckServerOptionsExist and every place it checks or inserts option collection records for a guild. Then trace the bot-invitation flow to assess whether records can be populated there. Done means the option-checking logic is decoupled from its current callers and the intended record-creation behavior is consistent.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
backend, database
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.