Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

add docs on EventBridge Scheduler #1368

Merged
merged 1 commit into from
Jul 15, 2024
Merged

add docs on EventBridge Scheduler #1368

merged 1 commit into from
Jul 15, 2024

Conversation

HarshCasper
Copy link
Member

No description provided.

Copy link

github-actions bot commented Jul 11, 2024

🎊 PR Preview has been successfully built and deployed to https://localstack-docs-preview-pr-1368.surge.sh 🎊

Copy link
Member

@joe4dev joe4dev left a comment

Choose a reason for hiding this comment

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

LGTM 👍

minor suggestion: It's a bit awkward that the tagging operation doesn't return any response, which might be misleading (i.e., perceived as error). So either skip it (because tagging is completely generic and unrelated to the service) or consider showing the result via awslocal scheduler list-tags-for-resource --resource-arn arn:aws:scheduler:us-east-1:000000000000:schedule/default/sqs-templated-schedule

@joe4dev joe4dev requested a review from maxhoheiser July 12, 2024 13:32
@HarshCasper HarshCasper merged commit cd72f8a into main Jul 15, 2024
3 checks passed
@HarshCasper HarshCasper deleted the Scheduler branch July 15, 2024 16:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants