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

Create + Update Operations Practice #34

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

LiorKGOW
Copy link
Owner

@LiorKGOW LiorKGOW commented Jan 5, 2023

Wanted to make sure the code in the API does the work properly :)

Create Actions: New & Create

Adding Guitar through a form
Adding Guitar through a form

Gutiar Added in the DB
Gutiar Added in the DB

Showing the DB from the Rails Console
Showing the DB from the Rails Console

Update Actions: Edit & Update

Editting form of new guitar in the correct URL
Update 1st - Editting form of  new guitar  in the correct URL

inserted data before submitting the form
Upsate 2nd - inserted data before submitting the form

Redirected to the correct URL of the updated Guitar (6)
Update 3rd - Redirected to the correct URL of the updated Guitar (6)

Back in the Gallery the updated Guitar is presented
Update 4th - Back in the Gallery the updated Guitar is presented

In Rails Console the Guitar was updated in the DB (yellow - before update, purple - after update)
Updated 5th - In Rails Console the Guitar was updated in the DB (yellow - before update, purple - after update)

@LiorKGOW LiorKGOW added the Do not Merge ❌ This PR is for a challenge but not for the final product label Jan 5, 2023
@LiorKGOW LiorKGOW self-assigned this Jan 5, 2023
@LiorKGOW LiorKGOW marked this pull request as draft January 5, 2023 10:06
@LiorKGOW LiorKGOW force-pushed the create-update-operations-practice branch from 68e4ff9 to 17ed793 Compare January 5, 2023 16:29
@LiorKGOW LiorKGOW force-pushed the create-update-operations-practice branch from 17ed793 to b8b88be Compare January 5, 2023 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do not Merge ❌ This PR is for a challenge but not for the final product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant