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

EC-134 Fix api token refresh #517

Merged
merged 2 commits into from
Apr 1, 2022
Merged

EC-134 Fix api token refresh #517

merged 2 commits into from
Apr 1, 2022

Conversation

@eliykat
Copy link
Member

@eliykat eliykat commented Mar 30, 2022

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Client PR for bitwarden/jslib#749.

Code changes

  • bw.ts - update dependencies, also extract the customUserAgent string into a separate variable for readability.

Testing requirements

Before you submit

  • I have checked for linting errors (npm run lint) (required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@eliykat eliykat changed the title Update dependencies Fix api token refresh Mar 30, 2022
@eliykat eliykat changed the title Fix api token refresh EC-134 Fix api token refresh Mar 30, 2022
@eliykat eliykat requested a review from differsthecat April 1, 2022 01:30
@eliykat eliykat marked this pull request as ready for review April 1, 2022 01:30
@eliykat eliykat merged commit 1e9a557 into master Apr 1, 2022
@eliykat eliykat deleted the fix/apikey-refresh branch April 1, 2022 01:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@differsthecat differsthecat differsthecat approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@eliykat @differsthecat