Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

Commit

Permalink
Lower Active alerts (#1789)
Browse files Browse the repository at this point in the history
When an alert is lowered it is not reflected in the IML GUI.

This is because we need to call `update_activity_health` in the
`Msg::RemoveRecord` variant.

Signed-off-by: Joe Grund <jgrund@whamcloud.io>
  • Loading branch information
jgrund authored Apr 16, 2020
1 parent 9b07385 commit cddf5bd
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 11 deletions.
22 changes: 11 additions & 11 deletions iml-gui/crate/Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 10 additions & 0 deletions iml-gui/crate/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -531,6 +531,16 @@ pub fn update(msg: Msg, model: &mut Model, orders: &mut impl Orders<Msg, GMsg>)
.proxy(page::filesystem::Msg::Stratagem)
.send_msg(stratagem::Msg::CheckStratagem);
}
warp_drive::RecordId::ActiveAlert(_) => {
let old = model.activity_health;
model.activity_health = update_activity_health(&model.records.active_alert);

orders.proxy(Msg::Notification).send_msg(notification::generate(
None,
&old,
&model.activity_health,
));
}
_ => {}
}
}
Expand Down

0 comments on commit cddf5bd

Please sign in to comment.