search

angeliklev / alis-ef-code-1st

3 stars 0 forks source link

Lms borrowers controller #61

Closed Moggers closed 11 years ago

angeliklev commented 11 years ago

All changes seems tiny and shouldn't break the rest of the application. I just... Can you create in AutomapperBootstrapper your own profile for Borrowers view models? They should nod belong to AssetEditModelProfile, shouldn't they?

Porleng commented 11 years ago

Your all validations are not working for me.

Moggers commented 11 years ago

Your all validations are not working for me. Which validations? And in what way are they not working?

Porleng commented 11 years ago

In your BorrowerExiryDate, it always shows "Please enter a valid date." even when I have correct format.

Porleng commented 11 years ago

Other things are working now. My Bad! Sorry

Porleng commented 11 years ago

There is another BorrowerController in root directory.

Moggers commented 11 years ago

Aye, when are we removing that? I believe Anzhelika holds the opinion that we should leave all that stuff there until the very end, and remove it all then.

If Anzhelika would give her opinion on this I could remove the borrowers and employees root example views and controllers.

Porleng commented 11 years ago

Ok then, I will talk to Anzhelika.

angeliklev commented 11 years ago

I asked the guys here in the class, nobody needs these old controllers along with their views in the root. Could you please delete them from the solution?

Moggers commented 11 years ago

Okay if that's it I'm going to go ahead and merge now.