JonAbrams / synth

The first back-end framework specially designed for single-page web applications
http://www.synthjs.com
MIT License
969 stars 70 forks source link

Add a simple model check to front-app.js template #42

Closed rzmz closed 10 years ago

rzmz commented 10 years ago

for cases when preloadedData is existing but not populated with any data

JonAbrams commented 10 years ago

Can you give an example of when this would happen?

rzmz commented 10 years ago

This happens when I start off by visiting /posts/add in my app and then click on /posts It was related to issue #41 (https://github.com/JonAbrams/synth/issues/41#issuecomment-46778524)

JonAbrams commented 10 years ago

In that case, have the getAdd API handler return null instead of {}.

On Sunday, June 22, 2014, Rasmus Piho notifications@github.com wrote:

This happens when I start off by visiting /posts/add in my app and then click on /posts It was related to issue #41 https://github.com/JonAbrams/synth/issues/41 (#41 (comment) https://github.com/JonAbrams/synth/issues/41#issuecomment-46778524)

— Reply to this email directly or view it on GitHub https://github.com/JonAbrams/synth/pull/42#issuecomment-46790701.

rzmz commented 10 years ago

Thanks! Feeling mighty dumb right now... :)

On 23 June 2014 00:04, Jon Abrams notifications@github.com wrote:

In that case, have the getAdd API handler return null instead of {}.

On Sunday, June 22, 2014, Rasmus Piho notifications@github.com wrote:

This happens when I start off by visiting /posts/add in my app and then click on /posts It was related to issue #41 < https://github.com/JonAbrams/synth/issues/41> (#41 (comment) https://github.com/JonAbrams/synth/issues/41#issuecomment-46778524)

— Reply to this email directly or view it on GitHub https://github.com/JonAbrams/synth/pull/42#issuecomment-46790701.

— Reply to this email directly or view it on GitHub https://github.com/JonAbrams/synth/pull/42#issuecomment-46792910.

JonAbrams commented 10 years ago

Ha, no worries!

On Sun, Jun 22, 2014 at 3:43 PM, Rasmus Piho notifications@github.com wrote:

Closed #42.

Reply to this email directly or view it on GitHub: https://github.com/JonAbrams/synth/pull/42#event-133911610