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

As a reviewer, I should be able to question on a selected item in a design. #137

Open
kimoktm opened this issue Mar 31, 2013 · 16 comments
Open

Comments

@kimoktm
Copy link
Collaborator

kimoktm commented Mar 31, 2013

save to db
save the part related to q
Documentation
story sequence diagram & artifact
unit test

Success:
-Click on the + button and then click and drag and the typing area is visible
-Click on add button so the question will be saved.
-questions is visible on the selected part.
-added in the database.

Failure :
-Questions will not be visible on the selected part

  • Unauthorized user to add questions to this design.
    -Questions are not added in the database (connection time out)
@kimoktm
Copy link
Collaborator Author

kimoktm commented Apr 2, 2013

Please review the UML @ytarek

@kimoktm
Copy link
Collaborator Author

kimoktm commented Apr 2, 2013

Please check the DOC @amatar07

@kimoktm
Copy link
Collaborator Author

kimoktm commented Apr 2, 2013

Please check the Scenario @marwanelderwy

@kimoktm
Copy link
Collaborator Author

kimoktm commented Apr 2, 2013

Please check the code @ahmed-osama-saad

@ndomar
Copy link
Owner

ndomar commented Apr 3, 2013

@marwanelderwy @amatar07 @ahmed-osama-saad @ytarek we need to review faster than that!

@ahmed-osama-saad
Copy link
Collaborator

i am not a code reviewer

@ahmed-osama-saad
Copy link
Collaborator

i am the documentation reviewer for c2

@ahmed-osama-saad
Copy link
Collaborator

@maged918 is the code reviewer for c2

@marwanelderwy
Copy link
Collaborator

3amalt review embareh khalas w ghayart el label

Sent from my iPhone

On Apr 3, 2013, at 11:11 AM, ndomar [email protected] wrote:

@marwanelderwy @amatar07 @ahmed-osama-saad @ytarek we need to review faster than that!


Reply to this email directly or view it on GitHub.

@kimoktm
Copy link
Collaborator Author

kimoktm commented Apr 3, 2013

Please review the code @maged918

@maged918
Copy link
Collaborator

maged918 commented Apr 3, 2013

Where is the code or the pull request hooked to this?

@maged918
Copy link
Collaborator

maged918 commented Apr 3, 2013

@kimoktm Please check the extra files as discussed, and please note that your pull request can not be automatically merged.

@amatar07
Copy link
Collaborator

amatar07 commented Apr 3, 2013

@kimoktm for the documentation just add this format to any method in the controller and the models: https://groups.google.com/forum/?fromgroups=#!topic/megasoft-13/_094FvOc9TI

@maged918
Copy link
Collaborator

maged918 commented Apr 3, 2013

@kimoktm Code verified.

@amatar07
Copy link
Collaborator

amatar07 commented Apr 3, 2013

@kimoktm good

@marwanelderwy
Copy link
Collaborator

@kimoktm scenario done

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

6 participants