uprm-inso4117-2023-2024-s2 / semester-project-study-pet

semester-project-study-pet created by GitHub Classroom
1 stars 3 forks source link

[Doc Feedback] Fix Needs #241

Closed irsaris closed 6 months ago

irsaris commented 7 months ago

"You still mention the study pet APP in the needs, but just barely... There are two things going on here: 1) you're summarizing the needs (study-centric, effective, fostering supportive and engaging learning experience) and 2) you present a first idea on how you want to satisfy the needs (which, of course, belongs into the ideas section)."

Eliminate the mention of the app in the needs as that's part of our system to be and the our needs are those that the app will resolve once it finally exists.

Adhel-J commented 6 months ago

Please check if the mention of the app and its fixes are okay.

irsaris commented 6 months ago

@PinkSylvie can you check this? im not sure if it was fixed correctly...

PinkSylvie commented 6 months ago

They refer to the project but project cold easily be replaced for app. The domain isn't the project either. The domain is the situation in which something like our app would be needed

Adhel-J commented 6 months ago

@PinkSylvie @irsaris Check again when you are able

PinkSylvie commented 6 months ago

You erased the whole needs section instead of editing the details mentioning the app/project to comply

Adhel-J commented 6 months ago

The criteria states that you cannot mention the application that is needed to fix the problem

Adhel-J commented 6 months ago

"For example, in a project improving the situation with people having too many calendars "the tool should direct all the tasks and events into one single place" is not a need." This is taken from the criteria that the professor did

PinkSylvie commented 6 months ago

That doesnt mean erase everything unless you are going to replace it with something equally as susbtantial

Adhel-J commented 6 months ago

restored the previous needs, can you tell me what you think

PinkSylvie commented 6 months ago

I think what you had made the correct change of eliminating the mention of the app or project. Blending what you had with what was already there would make the needs perfect

Adhel-J commented 6 months ago

merged the two ideas, sorry for the long wait

PinkSylvie commented 6 months ago

Looks good! Great job!