In order to reduce calls to CostExplorer to a minimum, the cost monitoring status is optimistically set to true in case of success, instead of calling the GetCostMonitoringStatus again
This PR builds upon the work done with #195 . Even though #195 isn't merged yet, this is safe to merge as it is being protected by a feature flag.
Please note: copy is not definitive and hasn't been reviewed yet
Changes
add cost_monitoring and experimental feature flag
add button to enable cost monitoring
add commands and mutation to get the current status and activate cost monitoring
Screenshots
How Has This Been Tested?
manually, by mocking backend responses
manually, by integrating locally with #195 and checking that interactions work as expected
Description
This PR adds a button to enable cost monitoring for the current account. It handles the following scenarios:
In order to reduce calls to CostExplorer to a minimum, the cost monitoring status is optimistically set to
true
in case of success, instead of calling theGetCostMonitoringStatus
againThis PR builds upon the work done with #195 . Even though #195 isn't merged yet, this is safe to merge as it is being protected by a feature flag.
Please note: copy is not definitive and hasn't been reviewed yet
Changes
cost_monitoring
andexperimental
feature flagScreenshots
How Has This Been Tested?
PR Quality Checklist
react-i18next
library (useTranslation hook and/or Trans component), see an example herenpm run build
builds without any errorIn order to increase the likelihood of your contribution being accepted, please make sure you have read both the Contributing Guidelines and the Project Guidelines
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.