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

sql server desteği eklendi. #4

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

mzuvin
Copy link

@mzuvin mzuvin commented Dec 21, 2024

Interceptor sınıfındaki hataları düzelttim. sql serverda update olduğu zaman eski değerleri mevcut kodda yazmıyordu. servis isimlerinin sonunda ilgili db'nin ismini ekledim karışmasın diye. örnek bir web proje ekledim. DesignTimeDbContext için de dinamik appsettings kodu ekledim. eksik olarak auditlog sınıfı partial yapılabilir ve userid eklenip o an değişikliğin kimin yapıldığı eklenebilir.

@furkandeveloper furkandeveloper self-requested a review December 22, 2024 11:24
@furkandeveloper furkandeveloper self-assigned this Dec 22, 2024
@furkandeveloper furkandeveloper added the enhancement New feature or request label Dec 22, 2024
@furkandeveloper
Copy link
Owner

Harika görünüyor. Oldukça fazla değişiklik var. Hepsini kontrol etmek biraz zamanımı alabilir. Katkın için teşekkür ederim. En kısa zamanda ilgileneceğim. 🚀 @mzuvin

Copy link
Owner

@furkandeveloper furkandeveloper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Yanlış bir namespace tanımı var.
  • Mevcut kütüphane üzerinde değişiklik yapılmamalı. Diğer PR'lar da olduğu gibi mevcut yapı örnek alınıyor. SQL Server içindeki tanımlamalar postgre üzerindeki gibi olmalı.
  • SQL server için how_to_use dökümanı yazılmalı.

@mzuvin
Copy link
Author

mzuvin commented Dec 29, 2024

  • Yanlış bir namespace tanımı var. fix.
  • Mevcut kütüphane üzerinde değişiklik yapılmamalı. Diğer PR'lar da olduğu gibi mevcut yapı örnek alınıyor. SQL Server içindeki tanımlamalar postgre üzerindeki gibi olmalı. fix.
  • SQL server için how_to_use dökümanı yazılmalı. mevcut dökümana eklendi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants