search

angeliklev / alis-ef-code-1st

3 stars 0 forks source link

Lms borrowers controller #40

Closed Moggers closed 11 years ago

Moggers commented 11 years ago

Just some small management changes to the LMS/Borrowers use case and a datepicker formatting fix.

Razokur commented 11 years ago

ViewModels folder is missing / it's contents are not there. I checked your branch and there was no ViewModel folder.

Moggers commented 11 years ago

Aye, that's because I've moved my borrower/employee viewmodels into their respective areas, where they should be.

Moggers commented 11 years ago

If someone could check this again that'd be swell.

Razokur commented 11 years ago

Me and Porleng (Popo Lak) noticed a few things didn't work.

On Edit, borrower expiry date has month 00.

Datepicker isn't working, generates double year.

Moggers commented 11 years ago

Thanks Jon.

Moggers commented 11 years ago

Got the fixes in, finally. That should be everything, should be one happy camper. If anyone wants to give a quick once over to make sure I didn't botch a reference or mess up a commit they're more than welcome to, else I'm just going to merge this sucker into the master branch.

angeliklev commented 11 years ago

Seems fine for me. I just failed to solve the conflicts while merging locally but all the rest worked ok. It is a nice update, we need it.