Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Device manager - device details section #9169

Merged
merged 8 commits into from
Aug 11, 2022
Merged

Conversation

kerryarchibald
Copy link
Contributor

@kerryarchibald kerryarchibald commented Aug 10, 2022

PSG-644

Adds basic device details section with name and metadata. Not used yet, blocked by #9151, #9155
Screenshot 2022-08-10 at 16 45 19

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

This change is marked as an internal change (Task), so will not be included in the changelog.

@kerryarchibald kerryarchibald added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Aug 10, 2022
@kerryarchibald kerryarchibald requested a review from a team as a code owner August 10, 2022 16:14
Copy link
Member

@robintown robintown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good otherwise

Comment on lines 35 to 42
{ values: [
{ label: _t('Session ID'), value: device.device_id },
{
label: _t('Last activity'),
value: device.last_seen_ts && formatDate(new Date(device.last_seen_ts)),
},
],
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could stand to be a bit more readable

Suggested change
{ values: [
{ label: _t('Session ID'), value: device.device_id },
{
label: _t('Last activity'),
value: device.last_seen_ts && formatDate(new Date(device.last_seen_ts)),
},
],
},
{
values: [
{ label: _t('Session ID'), value: device.device_id },
{
label: _t('Last activity'),
value: device.last_seen_ts && formatDate(new Date(device.last_seen_ts)),
},
],
},

@kerryarchibald kerryarchibald enabled auto-merge (squash) August 11, 2022 08:36
@kerryarchibald kerryarchibald merged commit 09aade2 into develop Aug 11, 2022
@kerryarchibald kerryarchibald deleted the psg-644/device-details branch August 11, 2022 09:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants