Co-worker reported that the unwrap of the optional adapter run id triggered a panic for them.
Each run has a unique broker run id, and an optional, non-unique adapter run id. Use the broker run id instead.
Actually, the whole PageBuilder type is now useless, but I’ll tackle that separately.
Signed-off-by: Lars Wirzenius liw@liw.fi
refactor: drop the PageBuilder type
After various previous changes, the PageBuilder::build now only calls StatusPage::new, so there’s no point in the type. Drop it.
Also replace StatusPage::new with StatusPage::default, for better style.
Signed-off-by: Lars Wirzenius liw@liw.fi
Co-worker reported that the unwrap of the optional adapter run id triggered a panic for them.
Each run has a unique broker run id, and an optional, non-unique adapter run id. Use the broker run id instead.
Actually, the whole PageBuilder type is now useless, but I’ll tackle that separately.
Signed-off-by: Lars Wirzenius liw@liw.fi
refactor: drop the PageBuilder type
After various previous changes, the PageBuilder::build now only calls StatusPage::new, so there’s no point in the type. Drop it.
Also replace StatusPage::new with StatusPage::default, for better style.
Signed-off-by: Lars Wirzenius liw@liw.fi