Skip to content

Replace lambda destinations#39

Merged
muhamadto merged 2 commits intomainfrom
replace_lambda_destinations
May 15, 2023
Merged

Replace lambda destinations#39
muhamadto merged 2 commits intomainfrom
replace_lambda_destinations

Conversation

@muhamadto
Copy link
Copy Markdown
Owner

@muhamadto muhamadto commented May 15, 2023

Hello, Please Review My Pull Request!

  • Added asserj like classes for testing

  • In this project lambda is called synchronously which means destinations are not invoked. So, there were removed. I added a dead letter topic to receive failures executions.

✔️ Checklist

  • Describe what you did in the pull request description
  • Add Unit and Integration Tests - at least 80% unit tests for new code.
  • Added or updated documentation

muhamadto added 2 commits May 15, 2023 13:31
Signed-off-by: muhamadto <muhamadto@gmail.com>
In this project lambda is called synchronously which means destinations are not invoked. So, there were removed. I added dead letter topic to receive failures executions.

Signed-off-by: muhamadto <muhamadto@gmail.com>
@muhamadto muhamadto merged commit cfe2434 into main May 15, 2023
@muhamadto muhamadto deleted the replace_lambda_destinations branch May 15, 2023 08:00
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