Skip to content

Commit e587999

Browse files
authored
Fix formatting of **Maintenance chart** in site_ops files (#1009)
* Fix formatting of **Maintenance chart** in site_ops files * Add Deprecated status to maintenance chart * Deprecate site ops pages that are no longer supported
1 parent 28af8dc commit e587999

File tree

120 files changed

+172
-167
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

120 files changed

+172
-167
lines changed

source/maintenance_chart_checker.py

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
+--------------+-------------------------------+----------------+--------------------------------+
99
| Review Date | Working Group Reviewer | Release |Test situation |
1010
+--------------+-------------------------------+----------------+--------------------------------+
11-
| 2025-04-13 | reviewer | Release | Pass/Fail |
11+
| 2025-04-13 | reviewer | Release | Pass/Fail/Deprecated |
1212
+--------------+-------------------------------+----------------+--------------------------------+
1313
1414
The script generates a summary report showing:
@@ -57,6 +57,7 @@ class MaintenanceStatus(NamedTuple):
5757
is_empty: bool
5858
has_fail: bool
5959
has_pass: bool
60+
has_deprecated: bool
6061
file_path: str
6162

6263
def parse_maintenance_chart(content: str) -> MaintenanceStatus | None:
@@ -94,14 +95,15 @@ def parse_maintenance_chart(content: str) -> MaintenanceStatus | None:
9495
# Look for pass/fail in test situation column (last column)
9596
columns = [col.strip().lower() for col in first_data_row.split('|') if col.strip()]
9697
if not columns: # Empty row
97-
return MaintenanceStatus(True, False, False, "")
98+
return MaintenanceStatus(True, False, False, False, "")
9899

99100
# The test situation is in the last column
100101
test_situation = columns[-1] if len(columns) >= 4 else ""
101102
has_fail = 'fail' in test_situation.lower()
102103
has_pass = 'pass' in test_situation.lower()
103-
104-
return MaintenanceStatus(is_empty, has_fail, has_pass, "")
104+
has_deprecated = 'deprecated' in test_situation.lower()
105+
106+
return MaintenanceStatus(is_empty, has_fail, has_pass, has_deprecated, "")
105107

106108
def get_folder_path(path: str, start_path: str, depth: int) -> str:
107109
"""Get the directory path at specified depth relative to start_path."""
@@ -132,8 +134,10 @@ def scan_directory(start_path: str, depth: int) -> Dict[str, List[MaintenanceSta
132134
status.is_empty,
133135
status.has_fail,
134136
status.has_pass,
137+
status.has_deprecated,
135138
file_path
136139
))
140+
137141
except Exception as e:
138142
print(f"Error processing {file_path}: {e}")
139143

@@ -145,8 +149,8 @@ def print_report(results: Dict[str, List[MaintenanceStatus]]):
145149
print("================================\n")
146150

147151
# Print table header
148-
header = "| {:<30} | {:>8} | {:>8} | {:>8} | {:>8} |".format(
149-
"Folder", "Total", "Empty", "Passed", "Failed"
152+
header = "| {:<30} | {:>8} | {:>8} | {:>8} | {:>8} | {:>10} |".format(
153+
"Folder", "Total", "Empty", "Passed", "Failed", "Deprecated"
150154
)
151155
separator = "-" * len(header)
152156

@@ -160,9 +164,10 @@ def print_report(results: Dict[str, List[MaintenanceStatus]]):
160164
empty = sum(1 for s in statuses if s.is_empty)
161165
passed = sum(1 for s in statuses if s.has_pass)
162166
failed = sum(1 for s in statuses if s.has_fail)
167+
deprecated = sum(1 for s in statuses if s.has_deprecated)
163168

164-
row = "| {:<30} | {:>8} | {:>8} | {:>8} | {:>8} |".format(
165-
folder, total, empty, passed, failed
169+
row = "| {:<30} | {:>8} | {:>8} | {:>8} | {:>8} | {:>10} |".format(
170+
folder, total, empty, passed, failed, deprecated
166171
)
167172
print(row)
168173

source/site_ops/concepts/index.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ Site Operators: Concepts
33

44
.. tags:: site operator, concept
55

6-
Maintenance Chart
6+
**Maintenance chart**
77

88
+--------------+-------------------------------+----------------+--------------------------------+
99
| Review Date | Working Group Reviewer | Release |Test situation |

source/site_ops/how-tos/add-waffle-flag-for-user.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ Follow these steps to enable a waffle flag for a single user. This may be useful
1515
#. Refresh the page and see that the entered LMS User ID(s) now correspond to their LMS Usernames.
1616

1717

18-
Maintenance Chart
18+
**Maintenance chart**
1919

2020
+--------------+-------------------------------+----------------+--------------------------------+
2121
| Review Date | Working Group Reviewer | Release |Test situation |

source/site_ops/how-tos/configuring-site-wide-analytics-tools.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ Just like how Matamo is integrated we can use other analytics tools as well.
5151
This will only be applied to the pages served by LMS and would not be supported by different MFEs used by the platform. Hence, to introduce the same, you need to create a custom component and put it in a slot. You can have a look at the `frontend-footer-component <https://github.com/openedx/frontend-component-footer>`_.
5252

5353

54-
Maintenance Chart
54+
**Maintenance chart**
5555

5656
+--------------+-------------------------------+----------------+--------------------------------+
5757
| Review Date | Working Group Reviewer | Release |Test situation |

source/site_ops/how-tos/enable_public_course_content.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ including the following.
121121

122122
* The Open edX mobile apps do not support public course content.
123123

124-
Maintenance Chart
124+
**Maintenance chart**
125125

126126
+--------------+-------------------------------+----------------+--------------------------------+
127127
| Review Date | Working Group Reviewer | Release |Test situation |

source/site_ops/how-tos/google-analytics.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ MFE pages
9999
:alt: A screenshot of the source code showing where to locate the GA4 snippet.
100100

101101

102-
Maintenance Chart
102+
**Maintenance chart**
103103

104104
+--------------+-------------------------------+----------------+--------------------------------+
105105
| Review Date | Working Group Reviewer | Release |Test situation |

source/site_ops/how-tos/index.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ Site Operators: How-tos
99
*
1010

1111

12-
Maintenance Chart
12+
**Maintenance chart**
1313

1414
+--------------+-------------------------------+----------------+--------------------------------+
1515
| Review Date | Working Group Reviewer | Release |Test situation |

source/site_ops/how-tos/use-frontend-plugin-slots.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ for more about Frontend Plugin Slots and their usage.
4646

4747
:doc:`/community/release_notes/sumac/customizing_learner_dashboard`
4848

49-
Maintenance Chart
49+
**Maintenance chart**
5050

5151
+--------------+-------------------------------+----------------+--------------------------------+
5252
| Review Date | Working Group Reviewer | Release |Test situation |

source/site_ops/index.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ Open edX Site Operators
2121

2222

2323

24-
Maintenance Chart
24+
**Maintenance chart**
2525

2626
+--------------+-------------------------------+----------------+--------------------------------+
2727
| Review Date | Working Group Reviewer | Release |Test situation |

source/site_ops/install_configure_run_guide/analytics/index.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ is not recommended for use.
4141

4242

4343

44-
Maintenance Chart
44+
**Maintenance chart**
4545

4646
+--------------+-------------------------------+----------------+--------------------------------+
4747
| Review Date | Working Group Reviewer | Release |Test situation |

0 commit comments

Comments
 (0)