[ ] 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.
[x] My code doesn't contain extensive if-else nesting. I am checking negative conditions first.
[x] 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.
[x] 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.
[x] There are no business logic or queries in routes.
[x] I have not put queries in Blade templates.
[x] I have not put JS and CSS in Blade templates and have not put any HTML in PHP classes.
[x] I have removed all debug code.
Nuxt coding practices
[ ] My Vue Components names are short, describes what it's intended use is and is in PascalCase
[ ] All the components exclusive to the page are in the page's components folder
[ ] I have preferred using Composition API.
[ ] I have preferred using <script setup> tag instead of the setup().
[ ] I have used asyncData or fetch hook (avoid mounted or created hooks) to fetch any initial data that will make the component load later than page.
[ ] I have marked any component that does not have reactive elements as functional.
[ ] I have used key for any v-for used.
[ ] I have not used v-for and v-if in the same element.
[ ] I have declared props with camelCase but used in templates with kebab-case.
[ ] I have validated my props with good definition, and have set there default values
[ ] I have added a Readme.md file for each component I have created.
[x] All my page specific methods and variables are in the page's store.
Feature Merge Request Checklist
[x] 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?
[x] did you follow the proper naming conventions in your code?
[ ] have you commented on your code with proper explanation?
Task #576 - 2023-02-21> VaahCms>Site Settings>General-development Time invested: 04:00 format | Billable: 04:00 | Non-Billable: 00:00
Client's comment or requirement
Proof of your testing, video, or image link?
Testing video Url:-(https://img-v1.dev.getdemo.dev/screenshot/chrome_QdpkNFZsO0.mp4)
Self Review Checklist
Naming Conventions
meaningful
,singular
and haveController
suffix?singular
and named after the table they are modelling?snake_case
andplural
?snake_case
without model name?descriptive
about which table they are seeding?short
,descriptive
andcamelCased
?short
,descriptive
andsnake_cased
?Constants and Configs
Fat methods and CRUD
Eloquent
queries.Eloquent models
or inRepository
/Helper
classes.Repository
/Helper
classes.Single Responsibility Principle
.@todo
.Other practices
down
method that reverses operations performed byup
method.fillable
property of its model.Nuxt coding practices
short
, describes what it's intended use is and is inPascalCase
components
folder<script setup>
tag instead of thesetup()
.functional
.key
for anyv-for
used.v-for
andv-if
in the same element.props
withcamelCase
but used in templates withkebab-case
.props
with good definition, and have set there default valuesReadme.md
file for each component I have created.Feature Merge Request Checklist
feature
withdevelop
?npm run prod
afterrebase
withdevelop
branch in all yourModules
&Themes
? Notes, you have to reactivate modules and themes to publish assets.patch
(x.x.<update-this>
) versionconfig.php
andcomposer.json
of module & theme?comments
¬es
inwireframe
and verified that you have taken care of that?UI
must match withdesign
&wireframe
if available?data-wdio
tag tolinks
,forms
& allform inputs
likeinput
,select
?develop
exists in yourfeature
branch after rebase?feature
as per the client requirement?