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.

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

French support #50

Merged
merged 4 commits into from
Jan 19, 2021
Merged

French support #50

merged 4 commits into from
Jan 19, 2021

Conversation

TheRaphael0000
Copy link
Contributor

Hello,

I have made a few changes to add the French support for the solver and the ui.

For the dictionary I use this repository : https://github.com/hbenbel/French-Dictionary/
I had to remove the "accents" because when we play Scrabble in French there are no accents on the letters.

I didn't implement the definition yet since I don't know any source for it.

There is also a strange behavior with the flag in the settings' menu.
Does it bother you to see what the problem is ? I have really no clue of what it could be since I'm terrible in JS/Node.

Best.
Raphaël.

Copy link
Owner

@kamilmielnik kamilmielnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @TheRaphael0000, this is great 🚀

I made full French support a goal for 2.3.0: https://github.com/kamilmielnik/scrabble-solver/milestone/7


I didn't implement the definition yet since I don't know any source for it.

I've created an issue for it: #52
I can do it with some basic crawling - can you maybe recommend an online french dictionary with definitions I could use for it?


There is also a strange behavior with the flag in the settings' menu.
Does it bother you to see what the problem is ? I have really no clue of what it could be since I'm terrible in JS/Node.

No problem, I'll have a look at it and fix it after this PR is merged.
Can you just describe the issue a bit more or record a short video/screencast, please? :)


Please resolve conflicts and it's good to merge 🔥

@kamilmielnik kamilmielnik merged commit 8d6fa88 into kamilmielnik:master Jan 19, 2021
@TheRaphael0000
Copy link
Contributor Author

For the definition I did a little research, but I didn't find anything.

Here is the problem I'm having with the menu for the language.

image

@kamilmielnik
Copy link
Owner

Here is the problem I'm having with the menu for the language.

Fixed with 69eb55f - thank you! :)

@kamilmielnik
Copy link
Owner

2.3.0 with French has been released 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants