This repository has been archived by the owner on Oct 20, 2023. It is now read-only.
Feature/ambiguous data #49
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…d and to pull the display name if a link has been created Removed old unused files and changed search on addBeaconSelectOrSuggest to search both display name and beacon name in prep for PR Added comments to explain some code choices and have delete, edit and add links all working Fix link mutation for editing and deleting having issue with deleted comments deleting all a beacon's commands Added delete link mutation to the comment box code Add delete link mutation Merged in develop and then tried to create link - created link does not seem to have a command attribute commented out reference to redeye electron have begun work on deleting and editing link Add manual property for links preparring to integrate Austin's work on connecting links to commands Link creation mutation added Add name property to link About to start pulling in other stuff Stuck with select and got the menu looking closer to correct. Ready to schedule time to discuss next steps Have mostly functional without search using Select, about to look into popover as alternative Have a popover working adn am about to try select instead
GoldingAustin
force-pushed
the
feature/AmbiguousData
branch
from
587544a to
0dc4a5a
Compare
GoldingAustin
approved these changes
Jan 6, 2023
GoldingAustin
added a commit
that referenced
this pull request
Jan 19, 2023
* expand multiCommands, route to previous expanded command when collapsing the last command * update commandBreadcrumb beaconSelect to support expanding multi commands * be able to expand multi commands, with collapse all button * clean up * removed old expand/collapse button * Explicitly pkg-fetch before running yarn run release, to avoid issues observed building in docker on [some] intel macs. * Add a "release:all" test for CI * Fav a comment won't add user * Update build.yml Remove release step * Update build.yml Remove extra whitespace * Tests to verify command and beacon counts (#45) * Cypress/blue team test (#48) * upgrade cypress 11.0.1 remove txt files downgrade to cypress 10.11, 11 seems to be broken upgrade to cypress 11 change to chromium revert commandtypes test more blue team tests blue team tests blue team ci/cd command fix graphql test imports moving files to separate blueteam / redteam testing upgrade docker images, caching cypress upload raw logs fixture file, graphql upload campaign WIP test upgrade cypress to 10.11.0 * reduced number of parallelization (+10 squashed commits) Squashed commits: [bf87ede] going to skip select beacon, select host test [666d32f] forgot to uncomment [1553970] changing cache [351c86e] edit select beacon blue team test [b61831c] fix flaky select host and beacon tests [3cdc4d9] test [5274cc2] upgrade cypress, edit workflow [c668831] waits for flaky tests [8381d2e] workflow update [72a80f5] make project id public * reduce to 0 matrix * comment out whole workflow * upgrade cypress to 11.2.0 * fix broken blue team checkscreenshot test * Add error message to http server close (#51) Closes #50 * query full CommpandGroupIds list, then query paginated commandGroups * add useEffect for command/commentsBox scroll, temp queryCommands * Database Migrations (#54) * fixed isLoading / undefined query issue * Set database mode to dev_persist in parsing machine when not in production to prevent database from being cleaned on upload of log files (#56) * Feature/543 timeline show time on drag (#57) * Timeline grabber - render time below grabber when dragging/clicking * take grabber time label out of svg and restyle * Feature/ambiguous data (#49) * Added code to only show link display name box if a link is being added and to pull the display name if a link has been created Removed old unused files and changed search on addBeaconSelectOrSuggest to search both display name and beacon name in prep for PR Added comments to explain some code choices and have delete, edit and add links all working Fix link mutation for editing and deleting having issue with deleted comments deleting all a beacon's commands Added delete link mutation to the comment box code Add delete link mutation Stuck with select and got the menu looking closer to correct. Ready to schedule time to discuss next steps Have mostly functional without search using Select, about to look into popover as alternative Have a popover working adn am about to try select instead * Update global operators graphql query & mutation auth (#58) * Add custom auth middleware for global operator graphql query and mutation Update login form with feedback that creating a new user is disabled Closes #55 * Fix commandGroup default query * fix progressBar for no comments group * Update graphql models * Feature/343 timeline hover bar to see beacon data (#60) * Timeline bars onHover and onClick labels via popover * bar onHover/onClick state to ctrl labels, update route when clicking beacons * click to show beacon list with router update when clicking listed beacon * Use new mikro-orm checkMigrationNeeded method to prevent unnecessary migration attempts (#59) * docs(changeset): Exposing BeaconMeta source LogLine * fix comments tab - command explanding/collapsing bug * change to headed cypress * Make migration imports static for pkg build (#66) * Rename Tokens - Light Theme Prep (#67) * add build alias to package scripts * REVERT THIS: disabled husky * updateTokenNames * rename and repair tokens * consolidate utility styles * rename styles.ts files * reorg client/ui-styles package * setting up theme vars * add theme switching toggle to settings * hide light theme so it doesn't end up in develop * Revert "REVERT THIS: disabled husky" This reverts commit 388b45a. Co-authored-by: James Bradford <j.bradford@pnnl.gov> * 0.8.2 Co-authored-by: Li, Huiqing <forqing@gmail.com> Co-authored-by: Matthew Galligan <matthew.galligan@cisa.dhs.gov> Co-authored-by: Yablargo <matthew.galligan@gmail.com> Co-authored-by: Courtney Carpenter <100231016+ccarpenter28@users.noreply.github.com> Co-authored-by: Huiqing Li <75810860+sharplessHQ@users.noreply.github.com> Co-authored-by: Sebastian Ang <22081109+sang2925@users.noreply.github.com> Co-authored-by: BrittDavis <brittany.f.davis@gmail.com> Co-authored-by: Devan Farrell <devanfarrell@users.noreply.github.com> Co-authored-by: James Bradford <arniebradfo@gmail.com> Co-authored-by: James Bradford <j.bradford@pnnl.gov>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🗣 Description
💭 Motivation and context
🧪 Testing
✅ Pre-approval checklist
in code comments.
to reflect the changes in this PR.
✅ Pre-merge checklist
✅ Post-merge checklist