AYS-333 | Solving test errors

[NNakreSS]

Checklist

Before submitting your pull request, ensure the following:

• Change Type:

• Bug fix

• Feature addition

• Documentation update

• Performance improvement

• Other (please specify)

• Summary:

  • Similar phone number error has been shown to the user in a more meaningful way, the problem of deleting the number of seats 0 has been fixed, the first error messages that appear in 2 steps for the application made on behalf of someone else have been fixed.

• Title and Branch Naming Conventions:

  • The pull request title follows the
    standard: Pull Request Naming Conventions.
  • The branch name follows one of the conventions outlined in
    the Branch Naming Conventions.

• Local Testing:

  • I have tested my changes locally on my browser, and they are working as expected.

• Code Quality:

  • The code is formatted according to the project's coding guidelines and style.
  • The code has been reviewed to ensure its quality.
  • The code does not contain any issues flagged by ESLint.

• Documentation:

  • Necessary documentation has been added or existing documentation has been updated.

• Reviewers and Assignees:

  • Default reviewers have been assigned to this pull request.
  • Assignees have been added if necessary.

• Related Issue:

  • AYS-333 | Emergency form added #62

• Labels and Associations:

  • No specific actions are required in the Labels and Associations section for this pull request.

[sanshigo345]

branch ve pr isimleri hiç açıklayıcı değil neyi review etmeliyim anlamadım, AYS-333 ticket'ı form hazırlanması ile alakalı olarak gözüküyor.

[NNakreSS]

branch ve pr isimleri hiç açıklayıcı değil neyi review etmeliyim anlamadım, AYS-333 ticket'ı form hazırlanması ile alakalı olarak gözüküyor.

Testteki hatalarını giderilmesi olarak yazdım, detaylar summary kısmında , header değil 😄

[sanshigo345]

branch ve pr isimleri hiç açıklayıcı değil neyi review etmeliyim anlamadım, AYS-333 ticket'ı form hazırlanması ile alakalı olarak gözüküyor.

Testteki hatalarını giderilmesi olarak yazdım, detaylar summary kısmında , header değil 😄

anladım, ama AYS-333 bir feature ticket'ı iken bugfix olarak branch açılması, AYS-333 kodu kullanılarak bütün ticketı değil küçük bir kısmını kapsayan branch/pr açmak bence yanlış. @agitrubard böyle durumlarda napılabileceğine dair bi açıklama yapsın

[NNakreSS]

branch ve pr isimleri hiç açıklayıcı değil neyi review etmeliyim anlamadım, AYS-333 ticket'ı form hazırlanması ile alakalı olarak gözüküyor.

Testteki hatalarını giderilmesi olarak yazdım, detaylar summary kısmında , header değil 😄

anladım, ama AYS-333 bir feature ticket'ı iken bugfix olarak branch açılması, AYS-333 kodu kullanılarak bütün ticketı değil küçük bir kısmını kapsayan branch/pr açmak bence yanlış. @agitrubard böyle durumlarda napılabileceğine dair bi açıklama yapsın

Testten dönen task , aynı kodu kullanmak zorundayız, aksi durum olmaz zaten , farklı bir issue değil.merge sonrası branch sildiğimiz için geriye dönük update de atamayız.

[sanshigo345]

neden localization yok?

[sanshigo345]

jfyi: nerden geldiğini tam çözemedim bi süre sonra bu hata geliyor

[NNakreSS]

neden localization yok?

Farklı issue açık zaten , institution daki gibi yapılamaz , burada olabildiğince sayfalar serverda tutulmalı , haliyle i18 işleyişi de farklı.

[NNakreSS]

jfyi: nerden geldiğini tam çözemedim bi süre sonra bu hata geliyor

deepl veya benzeri bir extension kullanıyor musun ?

[sanshigo345]

jfyi: nerden geldiğini tam çözemedim bi süre sonra bu hata geliyor

deepl veya benzeri bir extension kullanıyor musun ?

bazı extensionlar kullanıyorum ama deepl extension ne veya benzerleri neler bilmiyorum

[NNakreSS]

jfyi: nerden geldiğini tam çözemedim bi süre sonra bu hata geliyor

deepl veya benzeri bir extension kullanıyor musun ?

bazı extensionlar kullanıyorum ama deepl extension ne veya benzerleri neler bilmiyorum

Async çalışan extensionlarin yanıt vermemesi ya da çok geç yanıt vermesinden kaynaklı bir hata. Proje bağımsız, bunu görmezden gelebiliriz.

[agitrubard]

@NNakreSS Pull Request başlığının açık olması konusunda @sanshigo345'ya katılıyorum. Toplu bir çözüm uygulanmışsa bile şu formda bulunan hatalar giderildi gibi bir şey yazılabilir ✅

[NNakreSS]

@NNakreSS Pull Request başlığının açık olması konusunda @sanshigo345'ya katılıyorum. Toplu bir çözüm uygulanmışsa bile şu formda bulunan hatalar giderildi gibi bir şey yazılabilir ✅

Summary varken bunun başlıkta yazmasi ne kadar mantıklı bilemedim , başlık kısa ve öz olmalı sonuçta , summary kısmının amacı bu açıklamayı sağlamak.