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.

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

1350 IHE GW perf 2 #1674

Merged
merged 4 commits into from
Feb 28, 2024
Merged

1350 IHE GW perf 2 #1674

merged 4 commits into from
Feb 28, 2024

Conversation

leite08
Copy link
Member

@leite08 leite08 commented Feb 27, 2024

Ticket: https://github.com/metriport/metriport-internal/issues/1350

Dependencies

Description

  • perf adj on IHE GW configs
    • throw on errors
    • only one error processing per SOAP error
    • catch-all
  • minor DX updates

Testing

  • Local
    • PD/DQ/DR work
  • Staging
    • PD/DQ/DR work

Release Plan

  • Merge upstream
  • Merge this
  • Update IHE GW config

- notif about updates on IHE GW configs
- IHE GW readme info on init the package
- log on IHE GW script

Ref. metriport/metriport-internal#1350
@leite08 leite08 marked this pull request as ready for review February 27, 2024 23:17
Copy link
Member

@Goncharo Goncharo left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link
Member

@Goncharo Goncharo left a comment

Choose a reason for hiding this comment

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

:shipit:

@leite08 leite08 merged commit d4fb3cc into develop Feb 28, 2024
11 checks passed
@leite08 leite08 deleted the 1350-ihe-gw-perf_2 branch February 28, 2024 15:55
@leite08 leite08 mentioned this pull request Feb 28, 2024
16 tasks
Copy link

🎉 This PR is included in version 5.64.0-develop.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 5.64.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants