admin/ota-admin-portal
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

CEC-5694: display multiple hw_versions (#500)

Browse Source
This commit is contained in:
Tristan Timblin
2024-01-31 10:13:19 -08:00
committed by GitHub
parent 803d964a7e
commit e57d991ea8
2 changed files with 26 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/components/Manifest/Details/__snapshots__/index.test.jsx.snap
View File

@@ -116,7 +116,7 @@ Object {
ECUs:
<ul>
<li>
ICC, SW Version: 1000, HW Version: none, Mode: D
ICC, SW Version: 1000, HW Version: None, Mode: D
<br />
<ul>
<li>
@@ -171,7 +171,7 @@ Object {
ECUs:
<ul>
<li>
ICC, SW Version: 1000, HW Version: none, Mode: D
ICC, SW Version: 1000, HW Version: None, Mode: D
<br />
<ul>
<li>

42
src/components/Manifest/Details/index.jsx
View File

@@ -23,9 +23,8 @@ const ECUFileList = (files) => {
{files.map((file) => (
<li key={file.file_id}>
<a href={file.url}>{getFilename(file.url)}</a>
{`, File ID: ${file.file_id}, Type: ${file.type}, Download Size: ${
file.file_size
}, CRC32: ${file.checksum ?? "00000000"}
{`, File ID: ${file.file_id}, Type: ${file.type}, Download Size: ${file.file_size
}, CRC32: ${file.checksum ?? "00000000"}
${getMemoryRegion("Write", file.write_region)}
${getMemoryRegion("Erase", file.erase_region)}`}
</li>
@@ -35,20 +34,28 @@ const ECUFileList = (files) => {
};
const ECUList = (ecus) => {
if (!ecus || ecus.length === 0) return "None";
const none = "None";
if (!ecus || ecus.length === 0) return none;
return (
<ul>
{ecus.map((ecu) => (
<li key={ecu.name}>
{`${ecu.name}, SW Version: ${ecu.version ?? "None"}, HW Version: ${
ecu.hw_version ?? "none"
}, Mode: ${ecu.mode ?? "D"}${
ecu.self_download ? ", Self Download" : ""
}`}
<br />
{ECUFileList(ecu.files)}
</li>
))}
{ecus.map((ecu) => {
const name = ecu.name;
const version = ecu.version ?? none;
const hardwareVersions = ecu.hw_versions?.join(", ") ?? ecu.hw_version ?? none;
let mode = ecu.mode ?? "D";
if (ecu.self_download) {
mode += ", Self Download"
}
return (
<li key={ecu.name}>
{`${name}, SW Version: ${version}, HW Version: ${hardwareVersions}, Mode: ${mode}`}
<br />
{ECUFileList(ecu.files)}
</li>
)
})}
</ul>
);
};
@@ -104,9 +111,8 @@ const ManifestDetails = ({ opened }) => {
Hide Details
</div>
<ul>
<li>{`${manifest.name} ${manifest.version} [${
manifest.type || "standard"
}]`}</li>
<li>{`${manifest.name} ${manifest.version} [${manifest.type || "standard"
}]`}</li>
<li>{`${manifest.description} ${manifest.release_notes}`}</li>
<li>ECUs: {ECUList(manifest.ecu_updates)}</li>
</ul>