CivicTechFredericton / mealplanner

Primary Meal Planner repo
MIT License
9 stars 17 forks source link

[FEATURE] Template for mealplan #548

Open shanthisa opened 3 months ago

shanthisa commented 3 months ago

Is your feature request related to a problem? Please describe. Use the same favorite icon for saving it as template. The difference between the template and normal meal plan is that the users will not be assigned. I should be able to filter by templates.

Describe the solution you'd like As a meal designer/admin I should be able to create a meal plan as a template with no user assigned. I should be able to filter by templates in the search view.

Describe alternatives you've considered Favorites is the alternative, but does not make sense for meal plan Additional context Similar to favorites of meals in terms of implementation,.

shanthisa commented 3 months ago

For PR #551 - Please provide screenshots of before your changes i.e. current v2 and your UI of Create Meal Plan / Template. Looks like you have missed Tool tips. Also I need to see how is the spacing and UI.

shanthisa commented 3 months ago

Please make sure the user assigned is not visible in Edit Meal Template

ala-sk98 commented 3 months ago

QA done. Test case MP-Template-031: Passed

shanthisa commented 2 months ago

As a user, I am able to create a meal template. According to initial requirements, the user should not be able to. But since there is no harm in user creating a meal template, let's leave out the additional implementation required.

shanthisa commented 2 months ago

@Ala - Test case needs to change to reflect that it is now for any user. Ideally a negative test case should also be written as part of this issue and we could have got this issue earlier on.

ala-sk98 commented 2 months ago

@shanthisa Test case is now updated

shanthisa commented 2 months ago

@ala-sk98 Test case is wrongly saying it as meal section. Please provide proper navigation steps, screenshot, test data.

olamakhlouf commented 1 month ago

QA is done Test case updated