Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create,cancel, update a Special Interest Group for IEEE DTU #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Create,cancel, update a Special Interest Group for IEEE DTU #1

wants to merge 1 commit into from

Conversation

shubhi-sareen
Copy link

I am not sure about the correctness of the code, as I have coded in javascript for the first time. But I hope it gives a clear idea of what I wanted to do.

@sakshamsaxena
Copy link
Contributor

Hi @shubhi-sareen
I can't seem to lift your version. Is it working fine for you?

@shubhi-sareen
Copy link
Author

I have worked in Javascript for the first time. It would be of great help
if you could look into the code and tell me if my approach is correct. I
will look into the details of the code soon. Also, do you know any good
tutorials for using sails.js?

On 23 Jul 2016 14:08, "Saksham Saxena" [email protected] wrote:

Hi @shubhi-sareen https://github.com/shubhi-sareen
I can't seem to lift your version. Is it working fine for you?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARISPV-JqPg0tJzdNUTtCT7MUzDA4XKFks5qYdMIgaJpZM4JTSuI
.

@sakshamsaxena
Copy link
Contributor

Okay. It's throwing syntax error. Also, if you saw the Blueprint API, you'll see that there is a convention for naming the Controller and Model files. That's missing in your commit, because they're both named differently. I'm looking into the syntax part currently, so will let you know soon.
There aren't many tutorials on Sails itself, but you can get an idea of how a Node.js server works through Express.js tutorials.

@shubhi-sareen
Copy link
Author

Okay..Thanks!

On 23 Jul 2016 14:50, "Saksham Saxena" [email protected] wrote:

Okay. It's throwing syntax error. Also, if you saw the Blueprint API
http://sailsjs.org/documentation/reference/blueprint-api?q=blueprint-routes,
you'll see that there is a convention for naming the Controller and Model
files. That's missing in your commit, because they're both named
differently. I'm looking into the syntax part currently, so will let you
know soon.
There aren't many tutorials on Sails itself, but you can get an idea of
how a Node.js server works through Express.js tutorials.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARISPZ4OVVAQ1xed5QZK24kWbbNdmdMEks5qYdy_gaJpZM4JTSuI
.

@sakshamsaxena
Copy link
Contributor

Most probably your model isn't passing the automatic validation. Apart from that, RESTful operations mentioned in the Blueprint API are already defined, so you don't even need to write those in the controller. That's why I preferred Sails for production, because the common part is already implemented nicely.
Try to begin again on this. Otherwise the JS syntax itself was fine.
Read this answer on how to access models in controller in Sails.

@shubhi-sareen
Copy link
Author

Thanks

On Sat, Jul 23, 2016 at 3:37 PM, Saksham Saxena [email protected]
wrote:

Most probably your model isn't passing the automatic validation
http://sailsjs.org/documentation/concepts/models-and-orm/validations.
Apart from that, RESTful operations mentioned in the Blueprint API are
already defined, so you don't even need to write those in the controller.
That's why I preferred Sails for production, because the common part is
already implemented nicely.
Try to begin again on this. Otherwise the JS syntax itself was fine.
Read this answer http://stackoverflow.com/a/34084553/6428999 on how to
access models in controller in Sails.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARISPWiHCnOn_134DkTrVskak6s09whmks5qYefOgaJpZM4JTSuI
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants