Groups
Sign in
Groups
openSNP: development & operation
Conversations
About
Send feedback
Help
Can't create phenotype comments
0 views
Skip to first unread message
Philipp Bayer
unread,
May 6, 2015, 9:57:31 AM
5/6/15
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to snpr-dev...@googlegroups.com
Hi,
I guess due to the new "stricter" Rails 4 security stuff users currently
can't create comments.
The error is here:
http://opensnperr.herokuapp.com/apps/53faacfe1132c71f92000001/problems/55489604878204a56700006f
If I understand the new Rails correctly, all we need for the
PhenotypeComment controller is to add
def user_params
params.require(:user).permit(:phenotype_comment)
end
?
--
Helge Rausch
unread,
May 7, 2015, 2:18:48 AM
5/7/15
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to snpr-dev...@googlegroups.com
Fixed it:
https://github.com/gedankenstuecke/snpr/pull/163
Cheers,
Helge
XMPP+OTR:
helge....@jabber.ccc.de
Threema ID: XAY4STH9
signature.asc
Helge Rausch
unread,
May 7, 2015, 2:19:58 AM
5/7/15
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to snpr-dev...@googlegroups.com
Most of this is just reindenting the controller... ;) This looks better:
https://github.com/gedankenstuecke/snpr/pull/163/files?w=1
signature.asc
Reply all
Reply to author
Forward
0 new messages