Skip to content

Translate the code comments of the Hungarian examples to English - #6

Merged
hegedustibor merged 1 commit into
mainfrom
chore/english-code-comments
Oct 18, 2026
Merged

hegedustibor merged 1 commit into
mainfrom
chore/english-code-comments

Conversation

@hegedustibor

Copy link
Copy Markdown
Contributor

Mi változott

  • A magyar példafájlok (examples/hu/) kódblokkjaiban minden komment angol lett, mert a repóban a kódkommentek angolok. Ide tartoznak a //, # és {{-- --}} kommentek, valamint a konfigurációs példa többsoros /* | … */ blokkjai. A magyar szöveg és a magyar mintaadatok (nevek, címek, terméknevek) maradtak.
  • Két komment szándékosan magyar maradt: a hibakodok.md és a lekerdezes-es-sztorno.md példája a kivétel valódi, magyar nyelvű üzenetét mutatja.
  • A naplócsatorna kommentje mindkét nyelven pontosabb lett: null esetén nincs naplózás. Eddig azt írta, hogy ilyenkor az alapértelmezett csatornát használja, a config/simplepay.php szerint viszont kikapcsolja a naplózást.

Ellenőrzés

  • Szkripttel kigyűjtöttem a magyar példák kódblokkjainak összes kommentjét (92 darab). A két kimeneti példán kívül egyikben sincs magyar szöveg.
  • A nyitott Send the request fields the official SimplePay SDK sends #5 PR-rel próbamerge-elve ütközés nélkül egyesül, és együtt is minden dokumentált hívás egyezik az API-val.
  • Csak dokumentáció változott, kód és teszt nem.

Megjegyzések a review-hoz

  • A magyar konfiguracio.md és az angol configuration.md konfigurációs példájának kódja nem egyezik a valódi config/simplepay.php-val (például az api és az urls kulcsok). Ezt nem javítottam, mert nem fordítás.

- Code comments are English across the repository; the Hungarian example
  pages keep their Hungarian prose and sample data
- Keep the two comments that show the actual Hungarian exception message
- The log channel comment now says what null does: it disables logging
  (the Hungarian and English pages claimed the default channel is used)
@hegedustibor
hegedustibor merged commit c2d0ed2 into main Oct 18, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant