Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New observe by challenge and player views & follow by user in mks #27

Merged
merged 9 commits into from
May 9, 2022

Conversation

sei-jbritton
Copy link
Contributor

@sei-jbritton sei-jbritton commented May 6, 2022

Corresponding API changes: cmu-sei/Gameboard#33

  • Overhaul of previous observer dashboard to support two new ways of observing: by challenge and by team/player
  • Use of iframes to embed mks consoles into Gameboard UI instead of only viewing by new tab
  • Both "Observe Challenges" and "Observe Teams" show a table where each row can expand to show a grid of consoles
  • Consoles can be minimized, expanded, and opened in new tab
  • The mapping of which users are on which consoles will update frequently
  • Pinning rows to the top, searching, and sorting in the UI
  • In mks, you can specify a user id in the url to follow users to the current vm they are looking at with /mks/?f=0&o=1&u=<USER_ID> instead of /mks/?f=0&o=1&s=<CHALLENGE_ID>&v=<VM_NAME>

This is a major update to the observer dashboard to have more intuitive groupings of VMs. A major considering was designing it to prevent reloading of the embedded iframes when data updates or change detection happens (e.g. a user switches consoles or a team score changes). This is the reason that some processing is done in the browser to have more control over what is reloaded.

Copy link
Collaborator

@sei-mkaar sei-mkaar left a comment

Choose a reason for hiding this comment

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

Tested with Foundry Appliance 👍

@sei-mkaar sei-mkaar merged commit 889ab12 into main May 9, 2022
@sei-mkaar sei-mkaar deleted the observe-pr branch May 9, 2022 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants