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

metaadatok: angol oldal (?plang=en) megosztásakor legyen angol #71

Open
pisipite opened this issue Mar 27, 2023 · 2 comments
Open

metaadatok: angol oldal (?plang=en) megosztásakor legyen angol #71

pisipite opened this issue Mar 27, 2023 · 2 comments
Labels
pleasepleaseplease wontfix This will not be worked on

Comments

@pisipite
Copy link
Contributor

most a magyar jelenik meg:
kép
a jelenlegi kódban erre van lehetőség vagy kell hozzá fejleszteni?

@Cooty
Copy link
Collaborator

Cooty commented Apr 10, 2023

Sajnos ez a jelenlegi architechtúrával nem megoldható. Mindent kliens oldalon renderelünk (beleértve a fordításokat is, meg a lefordított meta adatokat is), amikor lejön a szerverről a sima index.html file ott fixen be van égetve a meta title magyarul, a Facebook (és az összes többi social media meg chat platform ami csinál ilyen link előnézetet) nem fogja lefutatni a JavaScriptet ami az egész alkalmazást megjeleníti csak azt nézi ami az elsődleges nyers HTML-ben van.

Az egyetlen lehetséges megoldásnak azt látom, ha az oldalt refaktorálnánk úgy, hogy minden oldalt előre statikusan legeneráljon. Ez megoldható, ha átültetjük az egészet mondjuk Next.js-re. Egyébként én már eljátszottam ezzel a gondolattal, sok előnye lenne SEO meg performancia szempontból is (a Hotel / Személy aloldalak biztosan gyorsabbak lennének amikor egy látogató direkt azokat nyitja meg és nem a Térképről navigál át), viszont ez nem egy triviális feladat, elég nagy időbefektetéssel járna. Becsült idő: nettó 1 hét munka, plusz mindent újra szépen ki kéne tesztelni, nehogy útközben eltörjünk valamit. Nem tudom megéri-e, ha találok rá időt szívesen belekezdek egy prototípus összerakásába (sok minden csak copy paste lenne, mert a Next.js is React-t használ, szóval nem vagyunk messze tőle), aztán meglátjuk belefutok-e valami nagyobb akadályba.

@pisipite
Copy link
Contributor Author

nagyon köszi, hogy utánajártál!
ez egyelőre nem prioritás, szóval akkor venném újra elő, ha valami miatt nagyobb szükség lenne a next.js -re.

@pisipite pisipite added the wontfix This will not be worked on label Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pleasepleaseplease wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants