Skip to content

Commit 47d86ee

Browse files
jr-rkclaude
andcommitted
Refactor: drop duplicate dspace eager-theme import in eager-themes.module.ts
The original VSB-TUO cherry-pick (#1288) left the dspace eager theme imported twice (as DSpaceEagerThemeModule and again, unaliased, as EagerThemeModule), with both listed in imports[]. Angular dedupes module imports so it was harmless, but it was cruft. Align the file with the other customer backports: dspace eager theme once + custom eager theme once. The custom theme stays eager (also keeps untyped-item theming working, ref DSpace#1897). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 747602f commit 47d86ee

1 file changed

Lines changed: 0 additions & 5 deletions

File tree

src/themes/eager-themes.module.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import { NgModule } from '@angular/core';
22
import { EagerThemeModule as DSpaceEagerThemeModule } from './dspace/eager-theme.module';
3-
import { EagerThemeModule } from './dspace/eager-theme.module';
43
import { EagerThemeModule as CustomEagerThemeModule } from './custom/eager-theme.module';
54

65
/**
@@ -18,10 +17,6 @@ import { EagerThemeModule as CustomEagerThemeModule } from './custom/eager-theme
1817
@NgModule({
1918
imports: [
2019
DSpaceEagerThemeModule,
21-
// Uncomment this because the `untyped-item` theming is not working when it is commented out.
22-
// Issue: https://github.com/DSpace/dspace-angular/issues/1897
23-
// Useful info in PR: https://github.com/DSpace/dspace-angular/pull/2262#issuecomment-1557146081
24-
EagerThemeModule,
2520
CustomEagerThemeModule,
2621
],
2722
})

0 commit comments

Comments
 (0)