search

openbmc / webui-vue

Web-based user interface built on Vue.js for managing OpenBMC systems
https://openbmc.github.io/webui-vue/
Apache License 2.0
56 stars 55 forks source link

Add Column "Context" on the session page #101

Closed kirankumarb07 closed 1 year ago

kirankumarb07 commented 1 year ago

Use Case

As a Web UI user, I need to be able to see the client ID and session ID on the sessions page.

In 2022.2, Redfish added support for the context parameter on the session resource. This parameter has the same function as the OemSession.ClientId field served. And we have the "Id" property from the redfish response. https://github.com/openbmc/bmcweb/commit/bb759e3aeaadfec9f3aac4485f253bcc8a523e4c

Since we have ID and context, I'm proposing the change to add the new column "context" on the sessions page as per the discussion with community members in the following PR.

https://gerrit.openbmc.org/c/openbmc/webui-vue/+/60919

Screenshot for reference:

image

Design Review Workflow

  1. Each design iteration will have a comment section
  2. The section will include:
    • A title with the iteration number
    • Any description or specific feedback the designer is requesting
    • Screenshots of the workflow
  3. Based on community and user feedback, we create a comment for the new iteration and repeat these steps
kirankumarb07 commented 1 year ago

Since the Client ID is moved as Context in bmcweb, added the Context column in WebUI, Modified the existing column "Client ID" to "Session ID" and mapped the "Id" property from the redfish response.

Note: 

The property "Context" is optional in redfish. If the Context holds any value it will display in WebUI as it is. If it is not having any value, "-" will be displayed in the WebUI.

PR: https://gerrit.openbmc.org/c/openbmc/webui-vue/+/60919

Screenshot for reference:

sessions

kirankumarb07 commented 1 year ago

Closing this task as PR#https://gerrit.openbmc.org/c/openbmc/webui-vue/+/60919 merged.