Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor: simplify project structure #15

Merged
merged 28 commits into from
Jun 25, 2022
Merged

refactor: simplify project structure #15

merged 28 commits into from
Jun 25, 2022

Conversation

zX3no
Copy link
Owner

@zX3no zX3no commented Jun 23, 2022 •
edited
Loading

  • Use traits to reduce code reuse.
  • Remove class methods.
  • Remove wrapper app class.
  • Remove configuration.
  • Simplify adding and removing file paths.
  • Move selected audio device to database.
  • Rework playlists.
  • Store volume in database.
  • Also, rewrite the database. I guess...

TODO:

zX3no added 28 commits June 22, 2022 18:06
testing new code structure
25bb8f8
search
69fde97
working queue
b648c42
status bar
e2d72c5
cleanup
6de59ea
playlists
227f073
arguments
94b7dd9
path mangement arguments
f6deb1e
argument cleanup
c94a63b
moved player out of queue and pedantic cleanup
3a62add
batch playlist
b48c9ee
test database
c22179d
working database
9161d55
removed duration from song
97a7114
fixed missing duration error
7741380
fixed wasted cpu cycles
9b5724c
search reworks
7b9ee35
working database
0b5c696
migrate to new database
cd53a69
dependency cleanup
4da3b6d
fixed list and rm arguments
1ae3935
working playlists
92567b7
store volume in database
ca607eb
cache songs in database
5807102
resume playback when adding new songs
4c1e593
store playback device in database
ddedd96
database refreshing
2407a08
fixed database crashes + cleanup
9b6c190
@zX3no zX3no merged commit ce50f21 into main Jun 25, 2022
@zX3no zX3no deleted the testing branch June 25, 2022 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Playlists Restore queue on relaunch
1 participant
@zX3no