You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Really love the concept, and loads of the functionality is there, I'm sure you're already aware of things that are not working so I won't bother commenting on them.
Something that came up for us in our project that I think you'll find interesting is also present in this code snippet in the model folder:
function createEntry(content, user_id) {
return create_entry.get({ content, user_id });
};
And here in the routes folder entries.js:
createEntry(entry, user_id);
By the way, they should maybe have different names. Anyway, I found a few bugs that we had from these values accidentally being put the wrong way round, and since they're being rewritten across different files I eventually found a cleaner way of working.
Really love the concept, and loads of the functionality is there, I'm sure you're already aware of things that are not working so I won't bother commenting on them.
Something that came up for us in our project that I think you'll find interesting is also present in this code snippet in the model folder:
And here in the routes folder entries.js:
By the way, they should maybe have different names. Anyway, I found a few bugs that we had from these values accidentally being put the wrong way round, and since they're being rewritten across different files I eventually found a cleaner way of working.
in the routes folder I think you should put:
Then in model:
It's not a big deal obviously, but just cleaner and less error-prone, imo.
The text was updated successfully, but these errors were encountered: