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

Feature/ays 609/add default value for has obstacle person exist #426

Conversation

m1erla
Copy link

@m1erla m1erla commented Jan 4, 2025

Checklist

Before submitting your pull request, ensure the following:

  • Title and Branch Naming Conventions:

  • Local Testing:

    • I have tested my changes locally on Postman, 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 SonarLint.
  • Documentation:

    • Necessary documentation has been added or existing documentation has been updated, specifically detailing changes made in Postman.
  • Testing:

    • Relevant unit tests have been written and included.
    • Relevant integration tests have been written and included.
  • Reviewers and Assignees:

    • Default reviewers have been assigned to this pull request.
    • Assignees have been added if necessary.
  • Labels and Associations:

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

@agitrubard
Copy link
Collaborator

Selamlar @m1erla ,

Bu pull request üzerinde bazı eksiklikler ve iyileştirilmesi gereken noktalar var, bu yüzden şimdilik PR’yi kapatıyorum. Aşağıdaki konulara dikkat ederek yeni bir pull request oluşturabilirsin:

  • Projeye zaten dahil olduğun için forklamadan çalışabilirsin. Projeyi clonelayarak kendi localinde çalışabilir ve feature branch’ini bu şekilde oluşturabilirsin. Bu, işlemleri daha sade ve standartlara uygun hale getirir.
  • Pull request başlığı standartlara uygun değil. Checklist'te yer alan ilk maddede yer alan Pull Request Naming Conventions dokümanına göz atarak başlığı uygun şekilde düzenleyebilirsin.
  • Assignees kısmında Pull Request sahibinin/sahiplerinin yer alması gerekiyor. Bu alanı eksiksiz doldurduğundan emin olabilirsin.
  • Checklist üzerinde bazı maddeler eksik ya da işaretlenmemiş. Her bir maddeyi tamamladığından ve kontrol ettiğinden emin olabilirsin. Bu, sürecin daha düzenli ilerlemesine yardımcı olur.

Tüm bu düzeltmelerden sonra yeni bir pull request açabilirsin. Yardıma ihtiyacın olursa BE kanallarından ekibe her zaman ulaşabilirsin 🙌🏼

@agitrubard agitrubard closed this Jan 4, 2025
@m1erla
Copy link
Author

m1erla commented Jan 4, 2025

Selamlar @m1erla ,

Bu pull request üzerinde bazı eksiklikler ve iyileştirilmesi gereken noktalar var, bu yüzden şimdilik PR’yi kapatıyorum. Aşağıdaki konulara dikkat ederek yeni bir pull request oluşturabilirsin:

  • Projeye zaten dahil olduğun için forklamadan çalışabilirsin. Projeyi clonelayarak kendi localinde çalışabilir ve feature branch’ini bu şekilde oluşturabilirsin. Bu, işlemleri daha sade ve standartlara uygun hale getirir.
  • Pull request başlığı standartlara uygun değil. Checklist'te yer alan ilk maddede yer alan Pull Request Naming Conventions dokümanına göz atarak başlığı uygun şekilde düzenleyebilirsin.
  • Assignees kısmında Pull Request sahibinin/sahiplerinin yer alması gerekiyor. Bu alanı eksiksiz doldurduğundan emin olabilirsin.
  • Checklist üzerinde bazı maddeler eksik ya da işaretlenmemiş. Her bir maddeyi tamamladığından ve kontrol ettiğinden emin olabilirsin. Bu, sürecin daha düzenli ilerlemesine yardımcı olur.

Tüm bu düzeltmelerden sonra yeni bir pull request açabilirsin. Yardıma ihtiyacın olursa BE kanallarından ekibe her zaman ulaşabilirsin 🙌🏼

@agitrubard Teşekkür ederim feedback için aslında ilk defa contribution yapıyorum ve ilk pull requestim. Bundan kaynaklı öğrenmeye çalışıyorum halen. Desteğiniz benim için çok önemli.

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