Skip to content

Conversation

Nathan-Gauci-Kasna
Copy link
Contributor

Similar to PR: #56
Docs have been regenerated for this change.

- Made event_trigger variable optional
(either trigger_http OR event_trigger should be supplied)
- Added trigger URL to outputs
@Nathan-Gauci-Kasna
Copy link
Contributor Author

@morgante please let me know if there are any issues with this PR or you need further info.

@github-actions
Copy link

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the Stale label Jun 28, 2022
Copy link
Member

@bharathkkb bharathkkb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR and sorry for the delay.

@bharathkkb bharathkkb removed the Stale label Jun 29, 2022
@comment-bot-dev
Copy link

@Nathan-Gauci-Kasna
Thanks for the PR! 🚀
✅ Lint checks have passed.

@bharathkkb bharathkkb merged commit 6e3f2d6 into terraform-google-modules:master Jun 30, 2022
@Nathan-Gauci-Kasna Nathan-Gauci-Kasna deleted the ng-trigger-http branch June 30, 2022 07:17
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.

3 participants