Repository: OWASP/owasp-mastg

Issue:
OWASP_MAS_Checklist issue
Hey,

Do you plan to release newer version of OWASP MAS Checklist?
I am asking because I have a strange issue.
In the current version when I select an answer in the first point for MASVS-STORAGE-1, the color is generated few positions above the row as presented in below screen.
Does someone have similar issue?

![checklist](https://github.com/OWASP/owasp-mastg/assets/58287037/7ecfc064-ea4c-4fd4-abc4-b68663befb8f)

OWASP_MAS_Checklist issue
Hey,

Do you plan to release newer version of OWASP MAS Checklist?
I am asking because I have a strange issue.
In the current version when I select an answer in the first point for MASVS-STORAGE-1, the color is generated few positions above the row as presented in below screen.
Does someone have similar issue?

![checklist](https://github.com/OWASP/owasp-mastg/assets/58287037/7ecfc064-ea4c-4fd4-abc4-b68663befb8f)



Discussion:
I have the same issue within every tab in the excel file. I repaired the document, but I have no idea where to do a pull request for a new version 😂
The excel file is 100 % automatically generated. Here are the relevant scripts

https://github.com/OWASP/owasp-mastg/blob/master/tools/scripts/yaml_to_excel.py

https://github.com/OWASP/owasp-mastg/blob/master/tools/scripts/excel_styles_and_validation.py

You can open a pull request by following these instructions: https://mas.owasp.org/contributing/3_PRs_and_Reviews/


@bl13pbl03p would you like me to assign the issue to you? 
I've assigned to you @bl13pbl03p based on your other message. Thanks for helping out!
@cpholguera I will do my best to fix it, but no guarantees can be given 😅. To confirm, all the rows that do not contain a L1, L2 or R value should not have a status column right? For example:

![Screenshot 2023-05-19 at 13 43 14](https://github.com/OWASP/owasp-mastg/assets/22095577/f7545ebe-9cff-4c46-87c3-edfa655d9787)

Exactly, only the "Status" column should have that. The others are reserved for the levels.
No worries if you can't make it in the end, thanks for trying it out!
I have the same issue within every tab in the excel file. I repaired the document, but I have no idea where to do a pull request for a new version 😂
The excel file is 100 % automatically generated. Here are the relevant scripts

https://github.com/OWASP/owasp-mastg/blob/master/tools/scripts/yaml_to_excel.py

https://github.com/OWASP/owasp-mastg/blob/master/tools/scripts/excel_styles_and_validation.py

You can open a pull request by following these instructions: https://mas.owasp.org/contributing/3_PRs_and_Reviews/


@bl13pbl03p would you like me to assign the issue to you? 
I've assigned to you @bl13pbl03p based on your other message. Thanks for helping out!
@cpholguera I will do my best to fix it, but no guarantees can be given 😅. To confirm, all the rows that do not contain a L1, L2 or R value should not have a status column right? For example:

![Screenshot 2023-05-19 at 13 43 14](https://github.com/OWASP/owasp-mastg/assets/22095577/f7545ebe-9cff-4c46-87c3-edfa655d9787)

Exactly, only the "Status" column should have that. The others are reserved for the levels.
No worries if you can't make it in the end, thanks for trying it out!

Resolving patch:
diff --git a/tools/scripts/yaml_to_excel.py b/tools/scripts/yaml_to_excel.py
--- a/tools/scripts/yaml_to_excel.py
+++ b/tools/scripts/yaml_to_excel.py
@@ -153,7 +153,7 @@ def create_security_requirements_sheet(wb):
         write_header(ws, category_title, mas_styles.MASVS_COLORS[group_id])
         set_columns_width(ws)
 
-        status_cells = 'H8:L400'
+        status_cells = 'I11:I400'
         ws.conditional_formatting.add(status_cells, mas_styles.rule_fail)
         ws.conditional_formatting.add(status_cells, mas_styles.rule_pass)
         ws.conditional_formatting.add(status_cells, mas_styles.rule_na)


Test patch: