AYS-415 | Environment config files added for production & development and used …

[gafur-demirci]

…for unvisible to EmergencyForm component

Checklist

Before submitting your pull request, ensure the following:

• Change Type:

• Bug fix

• Feature addition

• Documentation update

• Performance improvement

• Other (please specify)

• Summary:

  • Environment config files added for production & development and used for unvisible to EmergencyForm component.

• 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-415

• Labels and Associations:

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

[sanshigo345]

şu an sadece TEST ve PROD ortamı ayrımı olacağı için iki farklı .env dosyasına ihtiyaç yok. tek .env dosyası içinde yapılınabilir.

showEmergency çok abstract bir isimlendirme olmuş o daha açıklayıcı olabilir

[gafur-demirci]

showEmergencyForm ismi uygunsa bunu verebilirim, ek olarak önden hazırlanmış olması için prod env eklemiştim kalkması zorunlu ise silebilirim dosyayı, dönüşünüze göre düzenleyebilirim, teşekkürler.

[sanshigo345]

showEmergencyForm ismi uygunsa bunu verebilirim, ek olarak önden hazırlanmış olması için prod env eklemiştim kalkması zorunlu ise silebilirim dosyayı, dönüşünüze göre düzenleyebilirim, teşekkürler.

2 dosyanın olmaması şu an için daha uygun. onun yerine .env.example dosyasına sadece ortamın test olup olmadığını belirten bir variable eklenmeli. showEmergencyForm daha uygun evet

[gafur-demirci]

key değerini değiştirip env.production'ı sildim.

[sanshigo345]

key değerini değiştirip env.production'ı sildim.

2 dosyanın da olmaması şu an için daha uygun. onun yerine .env.example dosyasına sadece ortamın test olup olmadığını belirten bir variable eklenmeli. ona göre form gösterilmemeli veya gösterilmeli

[gafur-demirci]

env.example da yapılan tanımlamalar okunmuyor gibi görünüyor, onu da kaldırıp sadece env.development olması uygun mudur?

[sanshigo345]

hayır ekstra bir env dosyasına ihtiyacımız yok. example dosyası example olduğu için oradan okumaz.

.env dosyasına ve .env.example dosyasına test olduğunu belirten bi variable ekleyeceğiz. okuması gereken yer zaten .env dosyası olacak. variable ismi next conventionlarına uygun olacak ve TESTING_ENV ile bitebilir. ticketı yanlış hatırlamıyosam true ise form gözükücek.

[gafur-demirci]

.env file'a key ekleyip kullandım, TESTING_ENV='test' yapmak gerekiyorsa tekrar düzenleyebilirim.

[sanshigo345]

eline sağlık, ama bi kaç eksik nokta var ben düzenleyeceğim, sonra istersen konuşabiliriz yaptığım değişiklikleri