Skip to content

Commit cd38f6b

Browse files
arbrandesclaude
andcommitted
revert: sidebar widget registry refactor
Reverts the following two commits: - 59113b0 feat: move discussion topic prefetch from trigger to widget config lifecycle (#1897) - 0664dc3 feat: decouple notifications panel using widget registry mechanism (#1885) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 59113b0 commit cd38f6b

116 files changed

Lines changed: 10887 additions & 15693 deletions

File tree

Some content is hidden

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

docs/decisions/0010-upgrade-widget-extraction.md

Lines changed: 0 additions & 38 deletions
This file was deleted.

example.env.config.jsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,9 @@
11
import UnitTranslationPlugin from '@edx/unit-translation-selector-plugin';
2-
import { upgradeWidgetConfig } from './src/widgets/upgrade/src/index';
32
import { PLUGIN_OPERATIONS, DIRECT_PLUGIN } from '@openedx/frontend-plugin-framework';
43

54
// Load environment variables from .env file
65
const config = {
76
...process.env,
8-
SIDEBAR_WIDGETS: [upgradeWidgetConfig],
9-
107
pluginSlots: {
118
unit_title_plugin: {
129
plugins: [

jest.config.js

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -24,16 +24,20 @@ const config = createConfig('jest', {
2424

2525
// delete config.testURL;
2626

27-
// NOTE: jest-console-group-reporter@1.1.1 uses @jest/reporters@^30 internally
28-
// (via its peer dep resolution) but this project runs Jest 29, whose globalConfig
29-
// uses testPathPattern (string) not testPathPatterns (object). When any worker
30-
// exits uncleanly the reporter's SummaryReporter.onRunComplete crashes with
31-
// "Cannot read properties of undefined (reading 'isSet')", causing a non-zero
32-
// exit code even when all tests pass. Disabled until the package is updated for
33-
// Jest 29/30 compatibility.
34-
// config.reporters = [...(config.reporters || []), ["jest-console-group-reporter", {
35-
// afterEachTest: { enable: true, filePaths: false, reportType: "details" },
36-
// afterAllTests: { reportType: "summary", enable: true, filePaths: true },
37-
// }]];
27+
config.reporters = [...(config.reporters || []), ["jest-console-group-reporter", {
28+
// change this setting if need to see less details for each test
29+
// reportType: "summary" | "details",
30+
// enable: true | false,
31+
afterEachTest: {
32+
enable: true,
33+
filePaths: false,
34+
reportType: "details",
35+
},
36+
afterAllTests: {
37+
reportType: "summary",
38+
enable: true,
39+
filePaths: true,
40+
},
41+
}]];
3842

3943
module.exports = config;

0 commit comments

Comments
 (0)