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

Dökümanda bulunan ek parametrelerin eklenmesi #4

Merged
merged 5 commits into from
Jun 3, 2024

Conversation

sdkakcy
Copy link
Contributor

@sdkakcy sdkakcy commented Jun 3, 2024

Selamlar, öncelikle paket için teşekkürler ciddi bir iş yükünden kurtardı.

Katkıda bulunmak istememin ana sebebi iframe'i app diline göre açamıyor olmamdı. Buna göre dökümanda belirtildiği gibi lang parametresini ekledim. Elim değmişken timeout_limit parametresini de eklemiş oldum. Bunları yaparken parametreleri dökümandaki sırasına göre yeniden sıraladım. Bir de bir yorum satırı vardı sehven kalmış sanırım onu kaldırdım.

Son olarak ta biraz manuel formatlama yaptım. Bununla ilgili olarak projede bir .editorconfig ve styling için php cs fixer göremedim isterseniz bunları da ekleyebilirim ve farklı bir PR açabilirim.

@GizemSever GizemSever requested review from GizemSever and removed request for GizemSever June 3, 2024 21:36
@GizemSever GizemSever merged commit c3d9926 into GizemSever:main Jun 3, 2024
@GizemSever
Copy link
Owner

@sdkakcy Katkınız için teşekkür ederim 🙂

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.

2 participants