[NEW] [Device Management] Manage Devices Admin Table

[yash-rajpal]

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

[lgtm-com]

This pull request introduces 1 alert when merging 54cfc9c into 7814649 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

[dougfabris]

Awesome job here, congrats!
Some points to take a look at:

1. There's an overall issue related to TOTP when listing (sessions/list.all) and when getting info about the device (sessions/info.admin) it returning bad requests for some reason even when I set the TOTP auth, I would suggest taking a look on how it behaves without TOTP configured as well.

2. I'm missing the empty state when there are no devices. Additionally, I would suggest taking a look at the filter, the query is not filtering properly
   

[dougfabris]

I'm wondering if we could have a unique handleLoggout for the DeviceManagementInfo and for the DeviceManagementRow wdyt?

[lgtm-com]

This pull request introduces 1 alert when merging 4c106c8 into 602d107 - view on LGTM.com

new alerts:

• 1 for Useless conditional

[lgtm-com]

This pull request introduces 1 alert when merging 83d5bc1 into 602d107 - view on LGTM.com

new alerts:

• 1 for Useless conditional