[FOLIO-3878] Add ui-lists to snapshot hosted environments Created: 25/Aug/23 Updated: 15/Oct/23 Resolved: 13/Sep/23 |
|
| Status: | Closed |
| Project: | FOLIO |
| Components: | None |
| Affects versions: | None |
| Fix versions: | None |
| Type: | Task | Priority: | P2 |
| Reporter: | Matt Weaver | Assignee: | David Crossley |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original estimate: | Not Specified | ||
| Issue links: |
|
||||||||||||||||||||||||||||
| Sprint: | DevOps Sprint 160 | ||||||||||||||||||||||||||||
| Development Team: | FOLIO DevOps | ||||||||||||||||||||||||||||
| RCA Group: | TBD | ||||||||||||||||||||||||||||
| Description |
|
Hi DevOps, we'd like a new module (ui-lists - https://github.com/folio-org/ui-lists) added to the hosted environments for testing and demo purposes. |
| Comments |
| Comment by David Crossley [ 06/Sep/23 ] |
|
I tried to commence this task today. However the ModuleDescriptor is missing from the MD Registry. I see that Ryan removed that section from the GitHub Workflow. Please re-add and tell us when ready. |
| Comment by Matt Weaver [ 06/Sep/23 ] |
|
David Crossley, Ryan added the section back in and it ran successfully, so the MD should be good to go in the registry |
| Comment by David Crossley [ 07/Sep/23 ] |
|
I tried again today. Failure. See platform-complete/job/FOLIO-3878-refenvs-ui-lists/ [1m[31mERROR[39m[22m in [1m/home/jenkins/workspace/lete_FOLIO-3878-refenvs-ui-lists/node_modules/@folio/lists/src/pages/lists/ListPage.tsx[39m[22m
[1m./node_modules/@folio/lists/src/pages/lists/ListPage.tsx[39m[22m [1m[32m4:21-29[39m[22m
[1m[tsl] [1m[31mERROR[39m[22m[1m in /home/jenkins/workspace/lete_FOLIO-3878-refenvs-ui-lists/node_modules/@folio/lists/src/pages/lists/ListPage.tsx(4,22)
TS7016: Could not find a declaration file for module 'lodash'. '/home/jenkins/workspace/lete_FOLIO-3878-refenvs-ui-lists/node_modules/lodash/lodash.js' implicitly has an 'any' type.
Try `npm i --save-dev @types/lodash` if it exists or add a new declaration (.d.ts) file containing `declare module 'lodash';`[39m[22m
ts-loader-default_9df51fceb328e814
@ ./node_modules/@folio/lists/src/pages/lists/index.ts 1:0-27 1:0-27
@ ./node_modules/@folio/lists/src/pages/index.ts 1:0-35 1:0-35
@ ./node_modules/@folio/lists/src/index.tsx 4:0-86 10:77-85 11:101-120 12:106-118 13:96-110
@ ./node_modules/stripes-config.js
(I am away now on vacation, back my Tuesday. So this is stalled.) |
| Comment by David Crossley [ 12/Sep/23 ] |
|
Soon after my previous comment, Ryan added a fix to the pull-request platform-complete/pull/2466 However now the hourly build-platform-complete-snapshot builds are broken for an un-releated reason, so we cannot proceed until the responsible team fixes the problems (see Slack #hosted-reference-envs channel). |
| Comment by David Crossley [ 13/Sep/23 ] |
|
Finally the platform-complete CI is happening. So this is now merged. It will be in the folio-snapshot-2 build after today's run (and folio-snapshot 12 hours later). |