Closed ItsMurumba closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 86.85%. Comparing base (
c5990f5
) to head (076dd73
).:exclamation: Current head 076dd73 differs from pull request most recent head 8879c04. Consider uploading reports for the commit 8879c04 to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
[!WARNING]
Rate Limit Exceeded
@drizzentic has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 18 minutes and 21 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 2cc12e1ac1d1034ff10d2e5a93b073f39c96e782 and 8879c046a1be3d3c4bbd7c893c37df29ef499c5b.
The recent updates bring a cohesive integration of a new openhimConsoleBaseUrl
configuration setting across multiple configuration files, enrich the docker-compose.yml
with additional authentication environment variables, and expand the API capabilities by introducing a new endpoint for import maps. These modifications also encompass adjustments to test suites and constants to accommodate the new functionalities and configurations.
Files | Change Summary |
---|---|
.gitignore |
Added DockerfileARM to the ignore list. |
config/default.json config/development.json config/test.json |
Added openhimConsoleBaseUrl setting with the value "http://localhost:9000" . |
infrastructure/docker-compose.yml |
Updated with environment variables for authentication settings and OpenHIM console base URL. |
src/api/apps.js |
Introduced DEFAULT_IMPORT_MAP_PATHS import and added getTransformedImportMap for app data retrieval. |
src/constants.js |
Added OPENHIM_CONSOLE_BASE_URL constant and defined DEFAULT_IMPORT_MAP_PATHS . |
src/koaApi.js |
Included a new ImportMaps endpoint in the setupApp function. |
src/model/apps.js |
Removed Schema import from 'mongoose' and export of AppSchema . |
test/integration/appsAPITests.js |
Added imports and adjusted tests for getTransformedImportMap . |
🐇 In the code, new paths we chart, With URLs that play their part, Configs set and tests all primed, Docker's ready, all aligned.
A rabbit's joy in every line, For changes made, the code does shine. 🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Add capability to allow for the importmap to be enriched with new links for all external apps that do not exist in the monorepo but are already registered within openhim core.
The PR introduces an endpoint used to fetch transformed/enriched import map file
Summary by CodeRabbit
openhimConsoleBaseUrl
setting to various configuration files.docker-compose.yml
with new environment variables for authentication and OpenHIM console settings.