Skip to content

Conversation

addisonbeck
Copy link
Contributor

@addisonbeck addisonbeck commented Jul 20, 2020

Objective

Set up delete many w/admin methods for bulk deleting ciphers from within an org vault

Code Changes

Created relevant API methods and a stored procedure for implementing the objective

Related Issues

Relates to bitwarden/web #534
bitwarden/jslib PR #130 depends on this change

Copy link
Contributor

@cscharf cscharf left a comment

Choose a reason for hiding this comment

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

You will also need to create a script file for the DB update in util/Migrator/DbScripts following the convention there to drop + create the new stored procedure. This script is used when deploying the server AND as a schema update script for self-hosted instances upon update.

@addisonbeck
Copy link
Contributor Author

Added migration script

@addisonbeck addisonbeck requested a review from cscharf July 20, 2020 17:15
@CLAassistant
Copy link

CLAassistant commented Jul 21, 2020

CLA assistant check
All committers have signed the CLA.

@addisonbeck addisonbeck requested a review from kspearrin July 21, 2020 17:25
@addisonbeck addisonbeck requested a review from kspearrin July 21, 2020 21:58
kspearrin
kspearrin previously approved these changes Jul 22, 2020
@addisonbeck addisonbeck merged commit 229478a into master Jul 22, 2020
@addisonbeck addisonbeck deleted the Feature.Web.534.AllowMultiSelectInOrgVault branch July 22, 2020 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants