Skip to content

Input transformer for CloudWatch / EventBridge events #448

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Nov 25, 2021
Merged

Input transformer for CloudWatch / EventBridge events #448

merged 9 commits into from
Nov 25, 2021

Conversation

zirkelc
Copy link
Collaborator

@zirkelc zirkelc commented Jul 13, 2021

This PR adds the InputTransformer to CloudWatch and EventBridge events for issue #102

  • README updated
  • Test case added
  • Example serverless project added and deployed

@horike37
Copy link
Collaborator

@chriszirkel
Sorry for my delayed response. This is a great improvement! Thanks😊

Why have you put the example directory? I understand what it means. However, it needs more examples other than this PR if the directory is created, like covering almost all use cases of this plugin.

And can you fix the conflict?

@zirkelc
Copy link
Collaborator Author

zirkelc commented Oct 13, 2021

@horike37 the example directory was just for me to test it directly - I didn't mean to commit it.
I removed this directory and restored the package-lock.json from master.

@bayoudhi
Copy link
Contributor

bayoudhi commented Nov 24, 2021 •
edited
Loading

Hi @zirkelc any updates on this? It would be great if we can add this feature 🙂

@zirkelc
Copy link
Collaborator Author

zirkelc commented Nov 25, 2021

Hi @zirkelc any updates on this? It would be great if we can add this feature 🙂

I wasn't aware that there's an open issue that must be solved. I fixed the merge conflict, hope this PR is now ready to be merged :-)

@horike37
Copy link
Collaborator

@zirkelc
Thanks! It's perfect! This will be merged and shipped soon.

@horike37 horike37 merged commit cfaa125 into serverless-operations:master Nov 25, 2021
@horike37
Copy link
Collaborator

🎉 This PR is included in version 3.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@Alanfb Alanfb mentioned this pull request Feb 27, 2022
Merged
ss-betseqnzr pushed a commit to BetSEQNZR/serverless-step-functions that referenced this pull request Sep 8, 2023
@horike37
Merge pull request serverless-operations#448 from zirkelc/inputTransf…
13b1e28 
…ormer

Input transformer for CloudWatch / EventBridge events
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zirkelc @horike37 @bayoudhi