BULL-1717

Merge Request Checklist

Review

• Make sure the pipeline has passed. You may have to check the mirrored repo in code.il2.dso.mil to see the pipelines.
• Make sure that there are no errors in the development console.
• Make sure everything in the the ticket is addressed and that the acceptance criteria (if applicable) is met.
• Make sure all components and images match the design file or instruction given (if applicable).
• Make sure to take out any unused code.
• Make sure that common mistakes are not present in the code base. (See list under Common Mistakes below).
• Make sure the page is responsive (see the list for Responsiveness below for checklist).
• Check if any other issues may need to be checked (some examples can be found under Other items to look out for)

Common Mistakes

• Make sure all words are spelled correctly (if applicable).
• Make sure the right images are used (if applicable).
• Do not use any inline styles in the HTML component in files.
• Make sure everything is aligned correctly.
• Please avoid making global variables.
• Make sure error handling is in appropriate places.
• Remove all test code within the working sections when ready to push to master.

Responsiveness

For all below check the following breakpoints for Vue apps

Devices	Code	Screen sizes
Extra Small	xs	< 600px
Small	sm	600px > < 960px
Medium	md	960px > < 1264px
Large	lg	1264px > < 1904px
Extra Large	xl	> 1904px

• Check for bad word breaks within each breakpoint.
• Check to see if components are misaligned or have bad spacing, padding, or margin.
• Check to see if components overflow to either side of the viewport.
• Check if there are any weird visual glitches at different breakpoints.

Other items to look out for

• Make sure new API calls use the correct HTTP verb and when working with JSON make sure it is validated.
• With GETs, make sure that text/html return type is avoided and application/json is used when dealing with json data.
• Avoid transmission of confidental data in URL parameters and unnecessary parameters.
• Make sure all API calls are within a try catch block with appropriate error handling.
• If dealing with forms, make sure the form has input validation.
• Avoid logging confidental information.
• Make sure no-cache is on confidental web pages.
• Make sure rel="noopener noreferrer" is on any link that opens to a new tab.
• For browser performance, use .webp images instead of .png, .jpg, .jpeg, etc.

added 1 commit

• 8a065513 - Readded accidentally deleted target=_blank

Compare with previous version

marked the checklist item Make sure the pipeline has passed. You may have to check the mirrored repo in code.il2.dso.mil to see the pipelines. as completed

marked the checklist item Make sure that there are no errors in the development console. as completed

marked the checklist item Make sure everything in the the ticket is addressed and that the acceptance criteria (if applicable) is met. as completed

marked the checklist item Make sure all components and images match the design file or instruction given (if applicable). as completed

marked the checklist item Make sure to take out any unused code. as completed

marked the checklist item Make sure that common mistakes are not present in the code base. (See list under Common Mistakes below). as completed

marked the checklist item Make sure the page is responsive (see the list for Responsiveness below for checklist). as completed

marked the checklist item Check if any other issues may need to be checked (some examples can be found under Other items to look out for) as completed

marked the checklist item Make sure all words are spelled correctly (if applicable). as completed

marked the checklist item Make sure the right images are used (if applicable). as completed

marked the checklist item Do not use any inline styles in the HTML component in files. as completed

marked the checklist item Make sure everything is aligned correctly. as completed

marked the checklist item Please avoid making global variables. as completed

marked the checklist item Make sure error handling is in appropriate places. as completed

marked the checklist item Remove all test code within the working sections when ready to push to master. as completed

marked the checklist item Check for bad word breaks within each breakpoint. as completed

marked the checklist item Check to see if components overflow to either side of the viewport. as completed

marked the checklist item Check to see if components are misaligned or have bad spacing, padding, or margin. as completed

marked the checklist item Check if there are any weird visual glitches at different breakpoints. as completed

marked the checklist item Make sure new API calls use the correct HTTP verb and when working with JSON make sure it is validated. as completed

marked the checklist item With GETs, make sure that text/html return type is avoided and application/json is used when dealing with json data. as completed

marked the checklist item Avoid transmission of confidental data in URL parameters and unnecessary parameters. as completed

marked the checklist item If dealing with forms, make sure the form has input validation. as completed

marked the checklist item Make sure all API calls are within a try catch block with appropriate error handling. as completed

marked the checklist item Avoid logging confidental information. as completed

marked the checklist item Make sure no-cache is on confidental web pages. as completed

marked the checklist item Make sure rel="noopener noreferrer" is on any link that opens to a new tab. as completed

marked the checklist item For browser performance, use .webp images instead of .png, .jpg, .jpeg, etc. as completed

approved this merge request

added 7 commits

• 8a065513...f796e4d2 - 6 commits from branch master
• 960c2e9a - Resolved merge conflicts

Compare with previous version

approved this merge request

merged

mentioned in commit 148d7d7e