search

elixirschool / school_house

The new era of Elixir School now powered by @phoenixframework
https://elixirschool.com/
Apache License 2.0
155 stars 49 forks source link

Locale menu should validate current locale #87

Closed jaimeiniesta closed 3 years ago

jaimeiniesta commented 3 years ago

https://beta.elixirschool.com/klingon makes it think like we're supporting that language, but we're not :D

Captura de pantalla 2021-07-23 a las 11 29 38

kinson commented 3 years ago

😲 good catch! If you are not already working on a fix, I'll take a look this weekend. @doomspork my assumption is that we would want to redirect the user to / if they try to navigate to a locale that does not exist.

jaimeiniesta commented 3 years ago

@kinson please go ahead with this, I didn't plan on fixing it yet. Both redirecting to / or defaulting to the en locale sound good to me.