Skip to content
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.

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

feat: deprecate important property in favor of addToSummaryDashboard in CustomMetricGroup #271

Merged
merged 2 commits into from Nov 15, 2022

Conversation

Glyphack
Copy link
Contributor

@Glyphack Glyphack commented Nov 14, 2022

Resolves #242

To keep the add to summary naming consistent the new property has been added to CustomMetricGroup.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@Glyphack Glyphack changed the title Deprecate important property in favor of addToSummaryDashboard in CustomMetricGroup fix: Deprecate important property in favor of addToSummaryDashboard in CustomMetricGroup Nov 14, 2022
…n CustomMeticGroup

Resolves cdklabs#242 To keep the add to summary naming consistent the new property has been added to CustomMetricGroup.
Copy link
Member

@echeung-amzn echeung-amzn 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 contribution!

@echeung-amzn echeung-amzn changed the title fix: Deprecate important property in favor of addToSummaryDashboard in CustomMetricGroup fix: deprecate important property in favor of addToSummaryDashboard in CustomMetricGroup Nov 15, 2022
@echeung-amzn echeung-amzn changed the title fix: deprecate important property in favor of addToSummaryDashboard in CustomMetricGroup feat: deprecate important property in favor of addToSummaryDashboard in CustomMetricGroup Nov 15, 2022
@mergify mergify bot merged commit 727f0f2 into cdklabs:main Nov 15, 2022
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.

add_to_summary_dashboard does not work on monitor_custom
2 participants