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

[custom] Expose a position?: AxisPosition property at the metric level #283

Open
jedkass opened this issue Dec 6, 2022 · 1 comment
Open
Labels
feature-request New feature

Comments

@jedkass
Copy link

jedkass commented Dec 6, 2022

Feature scope

CloudWatch

Describe your suggested feature

Right now, the monitorCustom() method requires a metricGroups: CustomMetricGroup[] property. Each of these CustomMetricGroup objects has metric-group level graph configuration. However, each possible CustomMetric type in here does not seem to offer the ability to configure which axis position (i.e. right/left) a given metric should be displayed on. It looks like a property like this is exposed in CustomMetricWithAlarm, but it would be great to be able to configure axis positions for metrics in a dashboard that may not have associated alarms. I think it also makes more sense that axis position is associated with the metric, not alarm (but I could be missing something here)

@jedkass jedkass added the feature-request New feature label Dec 6, 2022
@jedkass
Copy link
Author

jedkass commented Jul 3, 2023

As a workaround, I've created a CustomMetricWithAlarm object with an empty addAlarm prop. For example:

{
      metric: metric,
      alarmFriendlyName: alarmFriendlyName,
      addAlarm: {},
      position: AxisPosition.RIGHT,
}

@echeung-amzn echeung-amzn changed the title Expose a position?: AxisPosition property at the metric level [custom] Expose a position?: AxisPosition property at the metric level Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature
Projects
None yet
Development

No branches or pull requests

1 participant