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

1839 Add endpoint sync facility w/ HIEs #2199

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

leite08
Copy link
Member

@leite08 leite08 commented Jun 3, 2024

Ref. metriport/metriport-internal#1839

Dependencies

none

Description

  • Add endpoint sync facility w/ HIEs.
  • add flag to enable/disable on specific HIEs - context

Testing

[Regular PRs:]

  • Local
    • [Indicate how you tested this, on local or staging]
    • ...
  • Staging
    • testing step 1
    • testing step 2
  • Sandbox
    • testing step 1
    • testing step 2
  • Production
    • testing step 1
    • testing step 2

[Release PRs:]

Check each PR.

Release Plan

[How does the changes on this PR impact/interact with the existing environment (database, configs, secrets, FFs, api contracts, etc.)?
Consider creating 2+ PRs if we need to ship those changes in a staged way]

[This is the release plan for production]

[You should execute the exact same steps when releasing to staging to validate it works]

[Add and remove items below accordingly]

  • ⚠️ Points to master
  • ⚠️ This contains a DB migration
  • Maintenance window scheduled/created at Checkly (if needed)
  • Execute this on ,
    • step1
    • step2
  • Added to monthly product update (deprecates a feature that needs to be communicated with customers)
  • Added to monthly product update (introduce a feature that would be useful customers)
  • Upstream dependencies are met/released
  • Release NPM packages
  • Fern Definition Updated
  • Release Fern SDKs
  • FFs have been set in Staging, Production, and Sandbox
  • No dependencies between API and Infra that will cause errors during deployment
  • [action n-1]
  • [action n]
  • Merge this

* @param facilityId
* @returns The updated facility.
*/
export async function syncFacilityWithinHIEs(cxId: string, facilityId: string): Promise<Facility> {
Copy link
Member Author

Choose a reason for hiding this comment

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

nit: within > with

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.

None yet

1 participant