Skip to content

feat: PEG-2146 more formats support and ccapi doc update #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
hmeng1024 wants to merge 12 commits into from
Closed

feat: PEG-2146 more formats support and ccapi doc update #34

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6dc4799
ccapi doc update (#1)
hmeng1024 Nov 14, 2024
e4a67b3
Merge branch 'mycloudnexus:main' into main
hmeng1024 Nov 14, 2024
618590d
Merge branch 'mycloudnexus:main' into main
hmeng1024 Jul 16, 2025
4e0d1bd
cleanup and wording update to satisfy Sonar check
hmeng1024 Jul 16, 2025
5b2c69f
remove unnecessary diff file
hmeng1024 Jul 16, 2025
277b6ca
PEG-2146
james-garrett Jul 16, 2025
ecef6b8
Updated spec file used
james-garrett Jul 17, 2025
b984d02
Revert "Updated spec file used"
james-garrett Jul 17, 2025
7c7a027
Removed console logs
james-garrett Aug 14, 2025
be29a20
Updated yarn lock
james-garrett Aug 15, 2025
158065d
Updated vulnerable packages
james-garrett Aug 15, 2025
4512ed6
Small amendments for code review requests and CI
james-garrett Aug 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ Use Docker Compose to build and run the container

```bash
docker compose up --build
SPEC_FILE=./specs/cc_api_uat_20241025_update.json docker compose up --build
```

## Usage
Expand Down
4 changes: 4 additions & 0 deletions compose.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,3 +18,7 @@ services:
image: consoleconnect/emulator:latest
ports:
- "9000:9000"
environment:
- SPEC_FILE=${SPEC_FILE}
volumes:
- ./specs:/app/specs
5 changes: 4 additions & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@
"devDependencies": {
"@eslint/js": "^9.14.0",
"@tsconfig/node20": "^20.1.4",
"@types/eslint-config-prettier": "^6.11.3",
"@types/eslint__js": "^8.42.3",
"@types/eslint-config-prettier": "^6.11.3",
"@types/express": "^4.17.21",
"@types/jest": "^29.5.14",
"@types/node": "^20.17.6",
Expand Down Expand Up @@ -38,9 +38,12 @@
"wait-on": "^8.0.1"
},
"dependencies": {
"@eslint/plugin-kit": "^0.3.4",

@jenni-okelly jenni-okelly Aug 18, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's behind us adding this as a dependency? Usually this package is for eslint plugins. It at least should definitely be a devDependency, but I would be very surprised if we should have this dependency at all. It's also locked to a very old version of this plugin (currently they are on major version 11).

"ajv-formats": "^3.0.1",
"date-fns": "^4.1.0",
"dotenv": "^16.4.5",
"express": "^4.21.1",
"form-data": "^4.0.4",
"openapi-backend": "^5.11.1",
"pino": "^8.21.0",
"pino-http": "^8.6.1",
Expand Down
Loading
Loading