Skip to content

build: Remove outdated sphinx-contentui package#126

Open
sarina wants to merge 3 commits intomainfrom
sarina/remove-scui
Open

build: Remove outdated sphinx-contentui package#126
sarina wants to merge 3 commits intomainfrom
sarina/remove-scui

Conversation

@sarina
Copy link
Copy Markdown
Contributor

@sarina sarina commented Apr 8, 2026

See openedx/docs.openedx.org#1417

This is an outdated package that was included in the docs cookie cutter. It can cause build errors and doesn't seem to be used.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍 WordPress Plugin Check Report

❌ Status: Failed

📊 Report

🎯 Total Issues ❌ Errors ⚠️ Warnings
5 5 0

❌ Errors (5)

📁 public/partials/openedx-commerce-public-display.php (1 error)
📍 Line 🔖 Check 💬 Message
0 missing_direct_file_access_protection PHP file should prevent direct access. Add a check like: if ( ! defined( 'ABSPATH' ) ) exit;
📁 utils/openedx-utils.php (1 error)
📍 Line 🔖 Check 💬 Message
0 missing_direct_file_access_protection PHP file should prevent direct access. Add a check like: if ( ! defined( 'ABSPATH' ) ) exit;
📁 includes/class-openedx-commerce.php (1 error)
📍 Line 🔖 Check 💬 Message
0 missing_direct_file_access_protection PHP file should prevent direct access. Add a check like: if ( ! defined( 'ABSPATH' ) ) exit;
📁 includes/class-openedx-commerce-loader.php (1 error)
📍 Line 🔖 Check 💬 Message
0 missing_direct_file_access_protection PHP file should prevent direct access. Add a check like: if ( ! defined( 'ABSPATH' ) ) exit;
📁 admin/partials/openedx-commerce-admin-display.php (1 error)
📍 Line 🔖 Check 💬 Message
0 missing_direct_file_access_protection PHP file should prevent direct access. Add a check like: if ( ! defined( 'ABSPATH' ) ) exit;

🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check

@sarina
Copy link
Copy Markdown
Contributor Author

sarina commented Apr 8, 2026

@feanil trying to remove this package everywhere but I think there's deeper issues here (it's saying I'm upgrading from Py 3.8 -> Py 3.12) - should I just close this until the packages are upgraded?

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.

1 participant