Page MenuHomePhorge
Files F32919649

D8984.1768219093.diff
No OneTemporary
Actions

Size
2 KB
Referenced Files
None
Subscribers
None

D8984.1768219093.diff
View Options

diff --git a/services/reports/src/email/template/mod.rs b/services/reports/src/email/template/mod.rs
--- a/services/reports/src/email/template/mod.rs
+++ b/services/reports/src/email/template/mod.rs
@@ -16,6 +16,30 @@
))
}
+// Examples:
+// [Android] Error report for User(ID = foo)
+// Media mission failed for User(ID = foo)
+// Thread inconsistency report for User(ID = foo)
+pub fn subject_for_report(
+ report_input: &ReportInput,
+ user_id: Option<&str>,
+) -> String {
+ let kind = title_for_report_type(&report_input.report_type);
+ let user = format!("User(ID = {})", user_id.unwrap_or("[unknown]"));
+ let object = if report_input.report_type.is_media_mission() {
+ media_mission_status(report_input)
+ } else {
+ "report"
+ };
+ let platform_prefix = if report_input.report_type.is_error() {
+ format!("[{}] ", report_input.platform_details.platform)
+ } else {
+ "".into()
+ };
+
+ format!("{platform_prefix}{kind} {object} for {user}")
+}
+
fn message_body_for_report(
report: &ReportInput,
report_id: &ReportID,
@@ -30,6 +54,8 @@
.to_string();
html! {
+ h2 { (title_for_report_type(&report.report_type)) " report" }
+ p { (intro_text(report, user_id)) }
ul {
li { "User ID: " b { (user) } }
li { "Platform: " (platform) }
@@ -39,6 +65,43 @@
}
}
+fn title_for_report_type(report_type: &ReportType) -> &str {
+ match report_type {
+ ReportType::ErrorReport => "Error",
+ ReportType::ThreadInconsistency => "Thread inconsistency",
+ ReportType::EntryInconsistency => "Entry inconsistency",
+ ReportType::UserInconsistency => "User inconsistency",
+ ReportType::MediaMission => "Media mission",
+ }
+}
+
+fn intro_text(report: &ReportInput, user_id: Option<&str>) -> String {
+ let user = format!("User (ID = {})", user_id.unwrap_or("[unknown]"));
+ match &report.report_type {
+ ReportType::ErrorReport => format!("{user} encountered an error :("),
+ ReportType::ThreadInconsistency
+ | ReportType::EntryInconsistency
+ | ReportType::UserInconsistency => {
+ format!("System detected inconsistency for {user}")
+ }
+ ReportType::MediaMission => {
+ let status = media_mission_status(report);
+ format!("Media mission {status} for {user}")
+ }
+ }
+}
+
+/// returns "success" or "failed" based on media mission status
+/// falls back to "completed" if couldn't determine
+fn media_mission_status(report: &ReportInput) -> &str {
+ report
+ .report_content
+ .get("mediaMission")
+ .and_then(|obj| obj["result"]["success"].as_bool())
+ .map(|success| if success { "success" } else { "failed" })
+ .unwrap_or("completed")
+}
+
impl Render for PlatformDetails {
fn render(&self) -> Markup {
let code_version = self

File Metadata

Mime Type
text/plain
Expires
Mon, Jan 12, 11:58 AM (17 h, 34 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
5923017
Default Alt Text
D8984.1768219093.diff (2 KB)

Event Timeline