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

User deletes his magazine S67 #100

Open
MinaHany94 opened this issue May 1, 2015 · 3 comments
Open

User deletes his magazine S67 #100

MinaHany94 opened this issue May 1, 2015 · 3 comments

Comments

@MinaHany94
Copy link
Collaborator

No description provided.

@mayarali
Copy link
Collaborator

mayarali commented May 2, 2015

@MinaHany94 You have to check first that this user is an admin in this magazine and you have to pass user id and magazine id and check that the magazine already exists

@MinaHany94
Copy link
Collaborator Author

@mayarali the user would not even be able to view the "Delete" link if he was not an admin in the magazine.The check is done in cancan and view level

@omaressameldin
Copy link
Collaborator

@MinaHany94
add alt boxes to check if the user is authorized to delete or not
actions are not sent from view to controller but from controller to models
params are sent from view to controller

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

No branches or pull requests

4 participants