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

Add support for PHP 8 #3

Open
hrrarya opened this issue Apr 23, 2022 · 1 comment
Open

Add support for PHP 8 #3

hrrarya opened this issue Apr 23, 2022 · 1 comment

Comments

@hrrarya
Copy link

hrrarya commented Apr 23, 2022

No description provided.

@etstaging
Copy link

QA Checklist

Reporter

  • Helpful title used  
    • Helpful title format: Scope(s) :: One Sentence To Describe The Issue
    • Example: VB :: Floating Toolbar :: Jumps To Top Of Page
  • Detailed description
    • Description should usually be 150+ characters.
  • Steps to reproduce included
  • Support Ticket URL included
  • Image and/or gif

Developer

  • Estimated points on issue
  • Steps to reproduce proven solved
  • PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
  • Proper RTL appearance

Peer Review

  • PR branch includes latest commits from master branch (at the time of checking)?
  • Steps to reproduce proven solved
  • Affected areas are tested and ok
  • Proper code documentation and conventions
  • Proper RTL appearance
  • i18n used if applicable
  • Test(s) included and coverage acceptable
  • Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
  • Database saved/updated? Data sanitized and validated?
  • Variable output properly escaped and validated?

Quality Assurance

  • All tests passed on CircleCI.
  • Steps to reproduce passed, the fix works.
  • No errors in console or output
  • Affected areas are tested and ok
  • The fix does not change a customer's website in any surprising or unintended way
    • If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

  • UI is to spec
  • UX is to spec
  • Design is to spec
  • No surprising visual changes to a customer's website will be experienced after updating
    • Any/all visual changes are migrated gracefully

Dev Review

  • Test(s) included and coverage acceptable
  • Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
  • Database saved/updated? Data sanitized and validated?
  • Variable output properly escaped and validated?
  • i18n used if applicable
  • Implementation acceptable, DRY, Scaleable, future safe
  • Best practices used, cleverness and grokage, ES6 use, etc.

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

No branches or pull requests

2 participants