Skip to content
This repository was archived by the owner on Jun 17, 2022. It is now read-only.

Set up API calls for bulk delete from within an org vault #130

Merged
merged 40 commits into from
Aug 4, 2020

Conversation

addisonbeck
Copy link
Contributor

@addisonbeck addisonbeck commented Jul 16, 2020

Objective

Create necessary API calls for bulk deleting ciphers from an org vault

Code Changes

Created relevant requests

Related Issues

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

@addisonbeck addisonbeck added the enhancement New feature or request label Jul 16, 2020
@addisonbeck addisonbeck requested a review from a team July 16, 2020 20:52
@CLAassistant
Copy link

CLAassistant commented Jul 21, 2020

CLA assistant check
All committers have signed the CLA.

}

getSelectedIds(): string[] {
return this.getSelected().map((c) => c.id);
Copy link
Member

Choose a reason for hiding this comment

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

I dont think this needs to be in jslib since it still only applies to the web vault. the org ciphers componenet already inherits from the regular ciphers component where this lives in the web repo so it should be reusable already.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Hinton and others added 12 commits July 23, 2020 13:32
* Initial work on windows hello support

* Switch to use windows.security.credentials.ui UserConsentVerifier

* Fix linting warnings

* Remove unessesary supportsBiometric from lock screen

* Rename biometric.main to windows.biometric.main. Add abstraction for biometric.

* Add support for dynamic biometric text.

* Add untested darwin implementation

* Rename fingerprintUnlock to biometric

* Add new functions to cliPlatformUtils.service.ts.

* Hide login if biometric is not supported

* Export default for biometric.*.main.ts

* Remove @nodert-win10-rs4/windows.security.credentials

* Add build requirements to readme

* Auto prompt biometric when starting the application.

* Ensure we support biometric before trying to auto prompt.

* Fix review comments and linting errors
* [SSO] Reset Master Password (#134)

* Initial commit of reset master password (sso)

* Updated line length error

* Updated import line again

* Added trailing comma

* restored reference data for RegisterRequest

* Updated tracking boolean name // added success route update based on passed boolean

* Added new API // reverted Register // deleted reset // added change pw and sso

* Changed redirect URI to protected to override in sub-class

* Updated api to setPassword // Updated request model name // Updated change password refs // Updated formatting

* Encoded necessary parts of authorize url // Added default catch error message

* Refactored methods inside change password base component // removed unnecesary query param for sso

* [lint] Fixed error (#137)

* Cleaned lint error

* Fixed sso lint error
cscharf and others added 3 commits August 1, 2020 16:28
…ire-nodert

try catch nodeRT module via require
* sso

* move break into try block

* make client id dynamic

* clientId is a string, DOH!

* reject if port not available

* lint fixes
@addisonbeck addisonbeck changed the title Move Cipher Selection Logic to Base Component Set up API calls for bulk delete from within an org vault Aug 4, 2020
@addisonbeck addisonbeck requested a review from kspearrin August 4, 2020 02:15
@addisonbeck addisonbeck merged commit 31a0be2 into master Aug 4, 2020
@addisonbeck addisonbeck deleted the Feature.Web.534.AllowMultiSelectInOrgVault branch August 4, 2020 21:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants