search

Starcounter / HelloWorld

Sample Starcounter application that shows CRUD operations with Palindrom (HTML+JSON-Patch client)
0 stars 7 forks source link

Excessive field in the Person class #53

Closed Mackiovello closed 7 years ago

Mackiovello commented 7 years ago

In https://github.com/StarcounterApps/HelloWorld/commit/e71662b057d07951a3e46865f94a1fa18f26f590, Spendings is defined and then never used. This is rather confusing. Spendings should instead be bound to the Expenses array in the view-model.

The issue now is that people try to use the workaround solution in the tutorial instead of using proper data bindings. I'll see if it's possible to simplify it.

Mackiovello commented 7 years ago

Created a solution in branch tutorial-20170504 Compare https://github.com/StarcounterApps/HelloWorld/commit/e71662b057d07951a3e46865f94a1fa18f26f590 to https://github.com/StarcounterApps/HelloWorld/commit/da908b8389ec9be4d2aff307eb6abdc516d8ac8f

It's significantly less code and much easier to explain. I'll integrate it into the tutorial and then close this issue.

Mackiovello commented 7 years ago

The simplified version is now live on the above-mentioned branch. It is now much easier to understand. The Expense Tracker page is now possible to follow. Closing