Skip to content

fix: Adjust import logic for integration_models to use --replace flag - #219

Open
nathaniel-itential wants to merge 1 commit into
itential:mainfrom
nathaniel-itential:fix/integration-model-replace-flag
Open

fix: Adjust import logic for integration_models to use --replace flag#219
nathaniel-itential wants to merge 1 commit into
itential:mainfrom
nathaniel-itential:fix/integration-model-replace-flag

Conversation

@nathaniel-itential

Copy link
Copy Markdown
Contributor

Description

Adds in the behavior required for the --replace flag that was already defined for integration models.

Type of Change

  • Bug fix
  • New feature
  • Documentation
  • Refactor
  • Chore

Changes Made

Adds an extra check for the replace flag in the import method for integration models to delete if existing before creation.

Testing

Tested in two cases, if the integration model exists and if it doesn't. Also checked to make sure non --replace calls error if the model already exists.

Checklist

  • Code follows the project's style guidelines
  • Self-review of code has been performed
  • Code has been commented where necessary
  • Tested with make setup or relevant profile
  • Commits follow conventional format (type: subject)
  • No secrets or credentials committed
  • Documentation has been updated accordingly
  • PR has been labeled appropriately (enhancement, bug, documentation, refactor, chore)

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.

1 participant