Device Provisioning UI

[Steve-Mcl]

PR Strategy

gitGraph
    commit
    commit
    branch device-provisioning-api
    checkout device-provisioning-api
    commit
    commit
    branch device-provisioning-ui
    checkout device-provisioning-ui
    commit
    commit
    checkout device-provisioning-api
    merge device-provisioning-ui
    checkout main
    merge device-provisioning-api
    commit

Loading

• Please review/approve/merge device-provisioning-ui before merging related PR Device provisioning API #1659

Description

Implement the UI portion of Provisioning Tokens management

• Add frontend API functions
• Add pages for managing provisioning tokens
• integrate new pages
• update audit log view for provisioning log entries

Related Issue(s)

#1212

Checklist

• I have read the contribution guidelines
• Suitable unit/system level tests have been added and they pass (see Device provisioning API #1659 )
• Documentation has been updated
  • Upgrade instructions
  • Configuration details
  • Concepts
• Changes flowforge.yml?
  • Issue/PR raised on flowforge/helm to update ConfigMap Template
  • Issue/PR raised on flowforge/CloudProject to update values for Staging/Production

Labels

• [-] Backport needed? -> add the backport label
• [-] Includes a DB migration? -> add the area:migration label

[knolleary]

Some initial wording changes proposed.

but also want to consider a broader renaming in the ui to standardise on 'register' rather than 'provision'.

[joepavitt]

Have made some improvements to UX:

• Target Snapshot included in the Provisioning Tokens table
• Team Settings to use side navigation tabs (consistent with Project & Device)
• Add slot into the "Register Device" dialog to provide more context as to the action available.

[Steve-Mcl]

@joepavitt @knolleary

Initial review comments addressed & tests fixed.

If there are further suggestions around wording, please let me know. Ta.

re-review requested.

[knolleary]

@Steve-Mcl added a comment 5 minutes ago

[joepavitt]

I'm happy with this as things stand - @knolleary you'v still got requested changes on your review, so I'll hold out from merging

[knolleary]

Looks like my comments have been actioned. I'm 👍🏻 on this now.

[joepavitt]

will wait for tests to run, then all good to go @Steve-Mcl