Skip to content

Game v2 #23

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
Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Game v2 #23

wants to merge 28 commits into from

Conversation

@aguschin
Copy link
Owner

No description provided.

@aguschin
Copy link
Owner Author

@NicolasPP I can't approve this PR since I created it, so you need to ask someone else to do that :)

@NicolasPP
Copy link
Collaborator

@NicolasPP I can't approve this PR since I created it, so you need to ask someone else to do that :)

Let me push my current changes before I merge the request

Nicolas and others added 23 commits October 12, 2023 16:08
added function to create colored text
d5165ed
showing both correct and incorrect guesses
abf7302
abstracted away some information from GameState to its own class Level
a557299
showing 4 lines of the song translated into emojis instead of song name
71ba73f
Merge branch 'main' into game_v2
1a74d66
added repr for Lyrics class
bde4f0c
made game_state into dataclass and removed repr
fd79ba6
Added support for the new json data.
bb35af0
showing the original lyrics when a guess is made
b658e4e
fixed bug where get songs was returning duplicate songs
d2724ad
fixed bug where result was showing the next level not current one
2cb3323
simplified get_songs function in DataManager
85ee90e
Merge branch 'main' into game_v2
8d95ccc
first working version on database integration
29dcee6
removed unused functions
9d72787
@aguschin
add pymongo, fix problem with requirement
66cc8fb
@aguschin
merge main into this
1718ac5
@aguschin
store top_300_spotify_translated.json in git
ff61ea0
improved game visuals
97d9f7d
Merge branch 'main' into game_v2
a189a97
Merge branch 'game_v2' of https://github.com/aguschin/lyrics2emoji in…
dcd87c7 
…to game_v2
@aguschin
Merge branch 'main' into game_v2
5d93ae4
@aguschin
use cleaned lyrics
8877fe1
@aguschin aguschin requested a review from NicolasPP October 23, 2023 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NicolasPP NicolasPP Awaiting requested review from NicolasPP

At least 1 approving review is required to merge this pull request.

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

@aguschin @NicolasPP