Skip to content

fix: pass invalidate option in delete_derived_resources - #745

Open
esguerrat wants to merge 1 commit into
masterfrom
fix/delete-derived-resources-invalidate
Open

esguerrat wants to merge 1 commit into
masterfrom
fix/delete-derived-resources-invalidate

Conversation

@esguerrat

Copy link
Copy Markdown
Contributor

Brief Summary of Changes

delete_derived_resources built its request params as a literal object containing only
derived_resource_ids[], so options like invalidate never reached the API. The option was
passed to call_api only as the options argument, which is used for auth, URL building
and timeouts, not as request params.

This change forwards invalidate with pickOnlyExistingValues, the same way
delete_derived_by_transformation already does.

What Does This PR Address?

  • GitHub issue (Add reference - #XX)
  • Refactoring
  • New feature
  • Bug fix
  • Adds more tests

Are Tests Included?

  • Yes
  • No

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