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

Unify Song Services #111

Merged
merged 203 commits into from
Mar 24, 2024
Merged

Unify Song Services #111

merged 203 commits into from
Mar 24, 2024

Conversation

AntonioMrtz
Copy link
Owner

Description

Unify Song services from master and streaming-master

Commit type

Feat

Issue

Solution

Proposed Changes

  • Adaptar documentacion
  • Envs y dockerfiles personalizados en carpeta other_archs
  • Adaptar frontend Player
  • Adaptar servicios canciones

Potential Impact

Tests Performed

  • [✅] Test performed

Screenshots

Additional Tasks

Assigned

@AntonioMrtz

@AntonioMrtz
chore : change token expire time to 7 days
a147050
@AntonioMrtz
fix : gap show all now is 14px
fbe29f2
@AntonioMrtz
feat : mostrarTodo user playlists
e97ca68
@AntonioMrtz
feat :add ShowAllSongsFromArtist fix ShowAllPlaylist
8996a95
@AntonioMrtz
feat : add mostrarTodo button UserProfile
92e19a0
@AntonioMrtz
fix : userprofile for artist fix canciones del artista header
8a74c8b
@AntonioMrtz
fix : reduce Playlist>Song artist hitbox
d85b6a3
@AntonioMrtz
fix : reduce cards artist hitbox
dfd3f89
@AntonioMrtz
feat : make SongCard use cards style and add clickable artist
258396f
@AntonioMrtz
fix : delete empty onContextMenu SongCard
3101ed8
@AntonioMrtz
fix : add repeat song increases play count by 1
dc28b9a
@AntonioMrtz
fix : disable song still playing while logout and play 2 songs at sam…
e1e59d1 
…e time
@AntonioMrtz
fix : alt text of image cards
fb3b50c
@AntonioMrtz
fix : no anidated button inside SongCard
2cf5bd8
@AntonioMrtz
file : move componentes/ to components/
e05dfc1
@AntonioMrtz
file : delete tech docs
82f2640
@AntonioMrtz
Create Procfile
8f7dcf4
@AntonioMrtz
Update Procfile
1a0e97a
@AntonioMrtz
Update Procfile
56f2e04
@AntonioMrtz
feat : add Dockerfile deploy Backend
6f56775
@AntonioMrtz
fix : add SECRET_KEY_SIGN env variable to example Dockerfile run
80ff081
@AntonioMrtz
fix : fix typo
ca40c1f
@AntonioMrtz
fix : modified path of mui imports
010900f
@AntonioMrtz
test : add test StartMenu
63b7a1c
@AntonioMrtz
chore : add github actions frontend tests
bb6c5ff
@AntonioMrtz
test : add RegisterMenu tests
00995eb
@AntonioMrtz
test : add UserProfile tests
6d72a0c
@AntonioMrtz
test : add Home tests
6fb9055
@AntonioMrtz
test : increase coverage Home test with mock fetchs
1b735f7
@AntonioMrtz
test : increase coverage UserProfile tests
4cd3549
@AntonioMrtz
fix : remove search by name artist from master-streaming migration
9b34f07
@AntonioMrtz
Merge branch 'master' of https://github.com/AntonioMrtz/SpotifyElectron…
3064296 
… into feat/Unify-Song-Services
@AntonioMrtz
refactor : rename mongodb-blob to db-blob secondary architecture
8c5fe7a
@AntonioMrtz
style : run black and isort
be44651
@AntonioMrtz
docs : complete style backend code with isort
9e1b285
@AntonioMrtz
chore : npm audit fix
ca0e39b
@AntonioMrtz
chore : delete unused init.py
09d645b
@AntonioMrtz
refactor : rename Player to PlayerStreaming
7b84738
@AntonioMrtz
feat : add lifespan handler to app start
af86a33
@AntonioMrtz
revert : remove lifespan_handler
07a3764
@AntonioMrtz
chore : remove unused comment
41cac1a
@AntonioMrtz
Revert "revert : remove lifespan_handler"
8f724bd 
This reverts commit 07a3764.
@AntonioMrtz
fix : on startup and teardown not deprecated
50778fb
@AntonioMrtz
feat : add PropertiesManager and its logic
fc9eb09
@AntonioMrtz
docs : update docs with new env variables
d920bbf
@AntonioMrtz
fix : backend test pipeline fix secrets
648da5e
@AntonioMrtz
feat : add dynamic song service depending on architecture
cda4e2e
@AntonioMrtz
style : run black and isort
812838d
@AntonioMrtz
ci : run streaming lambda and db blob tests on pipeline
0db5819
@AntonioMrtz
ci : divide backend tests pipeline by architecture
b5127a5
@AntonioMrtz
Revert "ci : divide backend tests pipeline by architecture"
b435c10 
This reverts commit b5127a5.
@AntonioMrtz
docs : add npm run build setup frontend
6f7bc98
@AntonioMrtz
test : add properties manager tests
20c4443
@AntonioMrtz
refactor : make protected PropertiesManager load_architecture and loa…
8f2c447 
…d_env_variables instead of private
@AntonioMrtz
feat : add PlayerFiles for DB-BLOB architecture
dc35051
@AntonioMrtz
test : fix properties manager tests by cleaning env in load arch no e…
e83ca32 
…nv provided
@AntonioMrtz
docs : frontend select Music Player depending on architecture
a6bb938
@AntonioMrtz AntonioMrtz merged commit 8daeada into master-streaming-lambda Mar 24, 2024
@AntonioMrtz AntonioMrtz deleted the feat/Unify-Song-Services branch March 24, 2024 15:43
@AntonioMrtz AntonioMrtz changed the title Feat/unify song services Unify Song Services Jul 2, 2024
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.
2 participants
@AntonioMrtz @actions-user