Skip to content

Conversation

@Thekid2002
Copy link
Contributor

No description provided.

Copy link
Contributor

@DaniekFjaes DaniekFjaes left a comment

Choose a reason for hiding this comment

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

A little birdie told me that this doesn't actually work, due to some issues with ef core, empty constructors and lists. So that should probably be fixed before we merge into master.

public void EditUser(User user, string firstName, string lastName, int phone, string email, string password, int? phoneNumber)
{
Console.WriteLine(user.GetType());
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Either an incomplete function or a misleading function name

Copy link
Contributor Author

Choose a reason for hiding this comment

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

// Parameterless constructor for EF Core
protected CustomerPurchaseOrder()
{
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Apparently, ef core doesn't approve :(

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.

7 participants