Skip to content

feat: add new destination s3-json

Description

  • added a new s3-json destination as an extension of the existing s3 destination
  • moved get_custom_config_properties() and get_generated_json_file_path() into sys_operation_classes.py, which is more suitable place for them
  • small refactor of s3 destination's send() method to keep the code cleaner with the addition of the new destination

How to test

Either locally run the script and check both - s3 and s3-json destinations, or setup propagation on devel, replace corresponding files on the machine and test through GUI.

Author's checklist

  • I have followed the contribution guidelines
  • This MR has been tested or does not change functionality
  • I have added relevant merge request dependencies (if this MR has any)
  • I have added the correct labels
  • I have assigned reviewers (if any are relevant)
  • I have edited the documentation (if the changes require it) or I have noted the need for the change if I do not have access to the documentation
  • I have marked all introduced BREAKING CHANGES or necessary DEPLOYMENT NOTES in the commit message(s)

Reviewer's checklist

  • This MR has been tested or does not change functionality
  • This MR has correct commit message format

Related issues

re STR-1581

Edited by Michal Berky

Merge request reports

Loading