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 reports a reply S64 #99

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

User reports a reply S64 #99

MinaHany94 opened this issue May 1, 2015 · 5 comments

Comments

@MinaHany94
Copy link
Collaborator

No description provided.

@MinaHany94
Copy link
Collaborator Author

@bassemmagdy @MohammedAnsary @AndrewMagdy @omarelhariry @mohabashraf

MinaHany94 added a commit that referenced this issue May 1, 2015
MinaHany94 added a commit that referenced this issue May 1, 2015
MinaHany94 added a commit that referenced this issue May 1, 2015
@mayarali
Copy link
Collaborator

mayarali commented May 2, 2015

@MinaHany94 reply id should be passed and the reply has to be checked if exists

MinaHany94 added a commit that referenced this issue May 2, 2015
@MinaHany94
Copy link
Collaborator Author

@mayarali fixed

@mayarali
Copy link
Collaborator

mayarali commented May 4, 2015

@MinaHany94 You should pass the ids as parameter and you should state the methods you are checking on or calling and don't use english words!

@bassemmagdy
Copy link
Collaborator

@MinaHany94 1)app/models/reply.rb
2) app/models/reportreply.rb
3)app/models/user.rb

These three models need documentation

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

3 participants