Sunshine: use sqlite db for persistent data
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 30/100
Research direction
Start by inventorying Sunshine's current JSON and Boost JSON persistence, then outline the proposed SQLite schema and migration approach. Done means persistent data is consolidated in SQLite, application IDs and migrations are supported, and database fixtures can cover unit-test scenarios.
Written by the indexing model from the issue text.
Description
Is there an existing issue for this item?
- I have searched the existing issues
Repositories
LizardByte/Sunshine
Languages/Skills/Technologies
C++, SQL
Description
I believe it's a better practice to store data in a proper database instead of random json files.
Pros
- All data in one location
- Apps can have an actual ID (auto incrementing)
- No more boost json
- Can use db fixtures for unit testing different scenarios
- Can use proper migrations when the db structure changes, storing the db version in migration table... with up and down migrations. For down migrations Sunshine could download the downgrade SQL file from the git repo since it wouldn't be available in the previous versions of Sunshine.
Cons
- All data in one location
- SQL
Estimated Effort
effort:Large
Priority
priority:Medium
Target Milestone
6+ months
Dependencies
No response
- Dominant language
- No language data
- Stars
- 2
- Forks
- 0
- Avg merge
- 25m
- Merged PRs (30d)
- 5
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from LizardByte/roadmap
-
1-3 months effort:Large LizardByte/Sunshine packaging planned priority:Medium
Difficulty 5/5 Over a week Newbie friendliness 35/100
LizardByte/roadmap#178 ·
-
1-3 months C++ CSS effort:Medium HTML JavaScript LizardByte/Sunshine planned priority:High Vue
Difficulty 5/5 Over a week Newbie friendliness 35/100
LizardByte/roadmap#153 · 2 reactions ·
-
Dependency Dashboard Opendependencies
LizardByte/roadmap#151 · 1 assignee ·
-
1-3 months Bash/Shell/Scripting effort:Medium GitHub actions LizardByte/actions planned priority:High
LizardByte/roadmap#134 · 1 assignee ·
-
1-3 months C++ effort:Large LizardByte/libdisplaydevice LizardByte/Sunshine LizardByte/tray planned priority:High
Difficulty 5/5 Over a week Newbie friendliness 25/100
LizardByte/roadmap#102 ·
All issues in LizardByte/roadmap
Similar issues
-
bug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
AXERA-TECH/ax-llm#77 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
games-on-whales/wolf#509 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100