search

ahmad2806 / Animals-shelter

"מערכת ניהול קשרי לקחות צער בעלי חיים"
MIT License
2 stars 3 forks source link

Check Team Project 2 Submission #1

Closed ahmad2806 closed 6 years ago

robi-y commented 6 years ago

Nice work. Please remove redundant text in issue text above.

Also, update the project description via repo. settings (it still say "A template repository with wiki for a team project at JCE's software engineering course")

Some other comments:

ahmad2806 commented 6 years ago

we will do as soon as possible . but we had a question about the time schedule that we set ..

https://github.com/ahmad2806/Animals-shelter/wiki/Inception-Planning

we tried to include every thing we need to do on it and we hope that the we can stick to Duration time that we sat ,but in Gantt chart we saw that we have a lot of TF ... do we need to worry about that ?. and maybe reschedule it again? ,or does it looks good for you (we need an expert opinion) thanx for checking this :)

On Fri, Mar 30, 2018 at 8:40 AM, Reuven Y. notifications@github.com wrote:

Nice work. Please remove redundant text in issue text above.

Also, update the project description via repo. settings (it still say "A template repository with wiki for a team project at JCE's software engineering course")

Some other comments:

  • Readme: link to live application is not ready (remove or comment out)
  • Inception: somewhat general risks not project related, a use case diagram in not optimal for discussing software structure, better use deployment or even class diagrams

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/ahmad2806/Animals-shelter/issues/1#issuecomment-377450444, or mute the thread https://github.com/notifications/unsubscribe-auth/Aji6F_crt4zbCuCn9QHu9j9nIQYb_bbfks5tjcU9gaJpZM4S558V .

robi-y commented 6 years ago

:+1: The plan is only an initial one, we will explore this more at week 5

On Fri, Mar 30, 2018, 4:58 PM ahmad2806 notifications@github.com wrote:

we will do as soon as possible . but we had a question about the time schedule that we set ..

https://github.com/ahmad2806/Animals-shelter/wiki/Inception-Planning

we tried to include every thing we need to do on it and we hope that the we can stick to Duration time that we sat ,but in Gantt chart we saw that we have a lot of TF ... do we need to worry about that ?. and maybe reschedule it again? ,or does it looks good for you (we need an expert opinion) thanx for checking this :)

On Fri, Mar 30, 2018 at 8:40 AM, Reuven Y. notifications@github.com wrote:

Nice work. Please remove redundant text in issue text above.

Also, update the project description via repo. settings (it still say "A template repository with wiki for a team project at JCE's software engineering course")

Some other comments:

  • Readme: link to live application is not ready (remove or comment out)
  • Inception: somewhat general risks not project related, a use case diagram in not optimal for discussing software structure, better use deployment or even class diagrams

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub < https://github.com/ahmad2806/Animals-shelter/issues/1#issuecomment-377450444 , or mute the thread < https://github.com/notifications/unsubscribe-auth/Aji6F_crt4zbCuCn9QHu9j9nIQYb_bbfks5tjcU9gaJpZM4S558V

.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/ahmad2806/Animals-shelter/issues/1#issuecomment-377529917, or mute the thread https://github.com/notifications/unsubscribe-auth/AA9nVzHACQHz_uk0sIPm5IcvDGxVXlqGks5tjjoNgaJpZM4S558V .