8000 Remove publication & fix health check by p-hoffmann · Pull Request #213 · OHDSI/d2e · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Remove publication & fix health check #213

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.

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

Merged
merged 5 commits into from
May 30, 2025

Conversation

p-hoffmann
Copy link
Collaborator

Merge Checklist

Please cross check this list if additions / modifications needs to be done on top of your core changes and tick them off. Reviewer can as well glance through and help the developer if something is missed out.

  • Automated Tests (Jasmine integration tests, Unit tests, and/or Performance tests)
  • Updated Manual tests / Demo Config
  • Documentation (Application guide, Admin guide, Markdown, Readme and/or Wiki)
  • Verified that local development environment is working with latest changes (integrated with latest develop branch)
  • following best practices in code review doc

Signed-off-by: Peter Hoffmann <954078+p-hoffmann@users.noreply.github.com>
@Copilot Copilot AI review requested due to automatic review settings May 29, 2025 09:02
Copy link
Contributor
@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the publications block from the JSON payload sent by load-demodatabase.mjs.

  • Deletes the publications array and its entries from the payload
  • Leaves only the vocabSchemas and authenticationMode properties
Comments suppressed due to low confidence (2)

scripts/load-demodatabase.mjs:175

  • Add or update automated tests to validate that the payload no longer contains the publications field and that downstream behavior remains correct.
    ],

scripts/load-demodatabase.mjs:177

  • Update any related documentation or README entries to reflect that the publications field is no longer supported in the payload.
"authenticationMode": "Password"

Signed-off-by: Peter Hoffmann <954078+p-hoffmann@users.noreply.github.com>
Signed-off-by: Peter Hoffmann <954078+p-hoffmann@users.noreply.github.com>
@p-hoffmann p-hoffmann changed the title Remove publication Remove publication & fix health check May 29, 2025
Signed-off-by: Peter Hoffmann <954078+p-hoffmann@users.noreply.github.com>
@p-hoffmann p-hoffmann merged commit cd43cf1 into release/v0.7.0-beta May 30, 2025
50 of 51 checks passed
@p-hoffmann p-hoffmann deleted the p-hoffmann/patch branch May 30, 2025 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0