Warning System

[adhamrait]

Description

Fixes issue #158 #159

Implements a warning system for errors that the user creates! Also, clicking on warnings will send the user to their location for easier fixing.

Still need to add unit/e2e tests for everything...

Pre-PR checklist

• [x] Ran npm run serve and:
  • [x] Checked basic functionality
  • [x] Checked that errors are handled
• [x] Ran npm run lint
• [x] Ran npm run test:unit
• [x] Ran npm run test:e2e and ran relevant tests
• [x] Attached reviewers to PR and pinged on Slack/email

Screenshots/GIFs

The warning pane is below the tool box/menu-bottom.

The warning pane includes a warning's name, its description, and hovering over shows its location. Clicking on the warning will send the user to the proper stunt sheet and select any dots that the warning involves.

The warning pane can also be filtered to only show warnings for the current stuntsheet.

Stuntsheets with errors have icons in MenuLeft

Major errors show up in red...

Demo in slack!

[adhamrait]

Add checks throughout flows for overlapping

[vhlee7]

Meeting notes: Demo?

warning1: stuntsheet different number of dots warning2: dots overlapping warning3: unnamed stuntsheet/dot warning4: stuntsheet w/ no dots errors in red: stuff that needs to be fixed errors not in red: stuff that does not need to be fixed

Stuff to work on: click on warning selects correct dots fix error naming - change error to issue *performance - iterate twice to find overlapping dots

• sort the dots by x-y coord and check for unique
• deep/shallow levels make the error display togglable check for overlapping dots in between flows

new issue: new unit test for shows