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

2.x feature -> 2.x develop | Updated: permission denied response #242

Merged
merged 168 commits into from
Jan 19, 2024

Conversation

we-alam-a001
Copy link
Contributor

@we-alam-a001 we-alam-a001 commented Jan 8, 2024

Task #528 - 2023-12-27 VaahCMS > Update Hint In Permission Denied Respose
Time invested: 11:00 format | Billable: 11:00 | Non-Billable: 00:00

Client's comment or requirement

Proof of your testing, video, or image link?

video link :- (https://img-v1.dev.getdemo.dev/screenshot/chrome_JCZYzGXB6n.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?

Constants and Configs

For a detailed information click here

  • API keys, Secret Keys, URL's etc., are stored as env variables.
  • I am accessing env variables via config.
  • I have used config and language files, constants instead of text in the code .
  • I have not hardcoded any value.

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.

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.
  • All my page specific methods and variables are in the page's store.

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.) 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?

we-vikram-wri231 and others added 30 commits December 30, 2023 11:51
Added: conditional if for language string


Fixed: login error while language is fr


Updated: extend controller for label


Fixed: to show bold text
…alization' into 2.x-feature/update-dashbaord-localization
we-vikram-wri231 and others added 29 commits January 19, 2024 13:57
…sponse' into 2.x-feature/permission-denied-response
@themodernpk themodernpk merged commit 9141cd0 into 2.x-develop Jan 19, 2024
1 check passed
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.

3 participants