Skip to content
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

Jump to bottom

Upgrade requirements and rename endpoints #54

Merged
merged 8 commits into from
Feb 20, 2025
Merged

Upgrade requirements and rename endpoints #54

merged 8 commits into from
Feb 20, 2025

Conversation

b-j-mills
Copy link
Collaborator

This fixes the tests and incorporates the endpoint renaming.

@mcarans
HDXDSYS-1754 Update HAPI Pipelines operational presence to read from …
27b8e92 
…HAPI dataset (#224)

* Read from HAPI operational presence dataset

* Read from HAPI operational presence dataset

* Fix admin test

* Update CHANGELOG
@coveralls
Copy link

coveralls commented Feb 13, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build 13424095530

Details

  • 88 of 93 (94.62%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 94.328%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/hapi/pipelines/database/hapi_dataset_uploader.py 70 75 93.33%
Totals Coverage Status
Change from base Build 13422326645: 0.3%
Covered Lines: 1530
Relevant Lines: 1622
💛 - Coveralls

@turnerm turnerm requested a review from danmihaila February 19, 2025 08:17
@danmihaila
Copy link

We will merge this after we deploy the new HAPI v2 version

@mcarans
HDXDSYS-1786 Update HAPI Pipelines HNO to read from HAPI dataset (#227)
e94051e 
* HAPI uploader

* Add cut down test data

* Update CHANGELOG
@danmihaila danmihaila merged commit a013cad into OCHA-DAP:main Feb 20, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexandru-m-g alexandru-m-g Awaiting requested review from alexandru-m-g

@danmihaila danmihaila Awaiting requested review from danmihaila

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@b-j-mills @coveralls @danmihaila @mcarans