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

Device Manager - add new labsed session manager screen (PSG-636)#9119

Merged
kerryarchibald merged 13 commits into
developfrom
psg-636/device-manager-labs-flag
Aug 8, 2022
Merged

Device Manager - add new labsed session manager screen (PSG-636)#9119
kerryarchibald merged 13 commits into
developfrom
psg-636/device-manager-labs-flag

Conversation

@kerryarchibald
Copy link
Copy Markdown
Contributor

@kerryarchibald kerryarchibald commented Jul 29, 2022

  • add labs flag for new device manager
  • add basic generic SettingsTab component (based on figma)
  • add stubbed 'Sessions' tab to user settings dialog

Includes #9118

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 Jul 29, 2022
@kerryarchibald kerryarchibald force-pushed the psg-636/device-manager-labs-flag branch from 73a9ad6 to 3dee437 Compare August 8, 2022 06:09
@kerryarchibald kerryarchibald marked this pull request as ready for review August 8, 2022 06:09
@kerryarchibald kerryarchibald requested a review from a team as a code owner August 8, 2022 06:09
Copy link
Copy Markdown
Contributor

@SimonBrandner SimonBrandner left a comment

Choose a reason for hiding this comment

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

Looks great!

Comment thread src/settings/Settings.tsx
Comment thread src/components/views/dialogs/UserSettingsDialog.tsx
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