feature -> develop | create wish list crud in store

[we-anil-y001]

[Task #462 - Vaah_store > Backend > Wish-list Crud]

(https://team.webreinvent.com/backend/collab/project/vaahcms/tasks/462)

Time invested: 09:05 format | Billable: 09:05 | Non-Billable: 00:00

Client's comment or requirement

Proof of your testing, video, or image link?

Old crud -> (https://img-v5.getdemo.dev/screenshot/wishlist.mp4)

New crud -> (https://img-v5.getdemo.dev/screenshot/chrome_aukNQk7KQp.mp4)

Self Review Checklist

Naming Conventions

For a detailed information click here

• Are your Controller names meaningful, singular and have Controller suffix?
• Are your Model names singular and named after the table they are modelling?
• Are your Table names snake_case and plural?
• Are your Table Column names snake_case without model name?
• Are your Seeder names descriptive about which table they are seeding?
• Are your method/function names short, descriptive and camelCased?
• Are your variable names short, descriptive and snake_cased?

Fat methods and CRUD

For a detailed information click here

• All requests are being validated
• I am not using raw SQL queries. I am using Eloquent queries.
• DB related logic are in Eloquent models or in Repository/Helper classes.
• I have not included business logic in Controllers. They are in Repository/Helper classes.
• My methods adhere to the Single Responsibility Principle.
• My code doesn't contain extensive if-else nesting. I am checking negative conditions first.
• I have removed all unwanted commented code.
• I have done my tech debt analysis and any unavoidable code leading to tech debt is marked with @todo .

Other practices

• All my migrations have down method that reverses operations performed by up method.
• Any new column added/altered in table has been also updated in the fillable property of its model.
• For seeders that are not populating sample data, it is not truncating table before seeding again.
• For seeders that are not populating sample data, the data is being populated from a separate external file.
• There are no business logic or queries in routes.
• I have not put queries in Blade templates.
• I have not put JS and CSS in Blade templates and have not put any HTML in PHP classes.
• I have removed all debug code.

Feature Merge Request Checklist

• have you rebased your feature with develop?
• have run npm run prod after rebase with develop branch in all your Modules & Themes? Notes, you have to reactivate modules and themes to publish assets.
• have you updated the patch (x.x.<update-this>) version config.php and composer.json of module & theme?
• have you read all the comments & notes in wireframe and verified that you have taken care of that?
• verify that the UI must match with design & wireframe if available?
• have you applied data-wdio tag to links, forms & all form inputs like input, select?
• did you verify the latest commit of develop exists in your feature branch after rebase?
• have you tested the feature as per the client requirement?
  /
• did you follow the proper naming conventions in your code?
• have you commented on your code with proper explanation?
  / # #
  /