Skip to content

Track expanded state of db items#1844

Merged
charisk merged 2 commits intomainfrom
charisk/track-expanded-state
Dec 7, 2022
Merged

Track expanded state of db items#1844
charisk merged 2 commits intomainfrom
charisk/track-expanded-state

Conversation

@charisk
Copy link
Copy Markdown
Contributor

@charisk charisk commented Dec 6, 2022

Adds logic to track expanded state for db items:

  • Adds an expanded property in the db config, which contains an array of items that are expanded
  • Adds an expanded property on some kinds of db items
  • Updates to db item creation to look at the expanded property in the db config and set the db item's expanded property where appropriate.
  • Adds expanded to a lot of test data

Checklist

N/A:

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@charisk charisk added the secexp label Dec 6, 2022
@charisk charisk requested review from a team as code owners December 6, 2022 16:19
@charisk charisk force-pushed the charisk/track-expanded-state branch from f2e0606 to da3be55 Compare December 6, 2022 16:27
Comment thread extensions/ql-vscode/src/databases/config/db-config.ts
@charisk charisk enabled auto-merge (squash) December 7, 2022 12:10
@charisk charisk merged commit d97eb2e into main Dec 7, 2022
@charisk charisk deleted the charisk/track-expanded-state branch December 7, 2022 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants