addon365 / b1ke-sh0wr00m

Apache License 2.0
0 stars 0 forks source link

Functionality to add Chit Schema #28

Closed sathishid closed 5 years ago

sathishid commented 5 years ago

Need to add Chit Schema UI, Service and Controller.

Refer ChitSchemaTable for details.

sathishid commented 5 years ago

Faced Many Issue: Branch, License & User Id are changed from Guid to int, which causes the test broken, NBuilder does not generate value to object type (Guid not generated) no issuee. But now NBuilder generate the branchId value which throws as it does not exists. So I have given the existing Branch.

Now causes new issue on using existing branchId. my releation is direct instead of having both id and virtual object but branch concept has vitual concept both combine raise issue. So I am revamping to both id and virtual concept to chit models.

Commit Chit e7b1c5341524d95001a318ded4cc104bca7252f4

As it has low priority, hold current status and continue to other task.

sathishid commented 5 years ago

Merged Master to current Branch:

Issues & Fixes after Merged Master: Issue 1:"Unable to cast object of type 'System.Decimal' to type 'System.Int32'." Solution: User table contains the id field as decimal in db and migration, So I have re runned the migration and issue fixed. Issue 2: Cannot convert to json VoucherInfo.Voucher.VoucherInfos Solution: There is recursive reference in VoucherInfo and Voucher, so I have workarounded by setting null to collection of VoucherInfo in each Vocher object Isssue 3: Account Book foriegn key issue Solution: AccountBook is in latest master branch, so considered AccountBook in Voucher by implemeting required service and initiated voucher with required account book.

Commit Details: a8ee25e1ae2eb89a9e675ffcad9742e5a6a51b8f