Skip to content

Conversation

mwaz
Copy link
Owner

@mwaz mwaz commented Jan 6, 2018

What does this PR do?

Add Status messages on deletion of both recipes and categories
Modify the API tables in README
Add methods to check for errors on invalid pages and methods

Description of Task to be completed?

Addition of status codes and meaningful messages on:

  • Logging in a user
  • Deleting a category (returns deleted category id)
  • Updating a Category
  • Deleting a Recipe (returns deleted recipe id)
  • Updating a Recipe
    Adding methods to handle invalid page requests and methods

How can this be manually tested?

Use postman or the online documentation to check for the messages raised when performing the above actions

What are the relevant pivotal tracker stories?

#154108647

@coveralls
Copy link

Coverage Status

Coverage increased (+0.008%) to 95.857% when pulling 8c66ec0 on fx-bug-fix-154108647 into a1f847f on ft-mashmallow-validation-154065039.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 95.911% when pulling 4a8787b on fx-bug-fix-154108647 into a1f847f on ft-mashmallow-validation-154065039.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 95.926% when pulling 866e8a8 on fx-bug-fix-154108647 into a1f847f on ft-mashmallow-validation-154065039.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 95.926% when pulling 25c35f5 on fx-bug-fix-154108647 into a1f847f on ft-mashmallow-validation-154065039.

@mwaz mwaz temporarily deployed to yummy-recipies-api January 8, 2018 11:58 Inactive
@mwaz mwaz temporarily deployed to yummy-recipies-api January 8, 2018 12:11 Inactive
@mwaz mwaz temporarily deployed to yummy-pi January 8, 2018 12:20 Inactive
@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 95.88% when pulling 17b0bb5 on fx-bug-fix-154108647 into a1f847f on ft-mashmallow-validation-154065039.

@mwaz mwaz merged commit d206537 into ft-mashmallow-validation-154065039 Jan 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants