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

Fixed issue for Multiple Schema FK. Added back removed code that gener… #239

Merged

Conversation

mwmccallum
Copy link
Contributor

Fixed issue for Multiple Schema FK. Added back removed code that generated additional annotations.

Test all pass locally after #236 correction. Northwinds Test DB does not have multiple schemas to build a test for #238 correction. However, I attached this project to my own database project that does have multiple schemas and the outstanding issue was fixed.

Closes #236
Closes #238

@mwmccallum
Copy link
Contributor Author

I forgot to mention that I implemented the EF7/8 HasTrigger functionality. Created method GenerateTrigger to generate the necessary text and it is called from GenerateEntityType, just after GenerateKey.

Generates:
entity.ToTable(e => e.HasTrigger ( "tr_acti_acno" ) );

Mike

@tonysneed tonysneed merged commit a449f1e into TrackableEntities:master Dec 30, 2023
@tonysneed
Copy link
Contributor

@mwmccallum Thank you very much for the PR which fixed these issues. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants