Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to dwn sdk js v0.2.3 #225

Merged
merged 13 commits into from
Oct 3, 2023
Merged

Upgrade to dwn sdk js v0.2.3 #225

merged 13 commits into from
Oct 3, 2023

Conversation

diehuxx
Copy link

@diehuxx diehuxx commented Sep 25, 2023

Upgrade the following packages to the latest dwn-sdk-js version

  • agent
  • user-agent
  • proxy-agent
  • api

Notable changes dwn-sdk-js

  1. Remove monkeypatches for creating DWN authorization signature
  2. Add filter to RecordsRead

@diehuxx diehuxx force-pushed the agent-dwn-sdk-js-v0.2.2 branch 3 times, most recently from 66c1d8d to 256fc69 Compare September 25, 2023 07:27
@diehuxx diehuxx force-pushed the agent-dwn-sdk-js-v0.2.2 branch from c8fd909 to 78c235f Compare September 26, 2023 22:33
@diehuxx diehuxx force-pushed the agent-dwn-sdk-js-v0.2.2 branch from 50bbde2 to f177e99 Compare September 28, 2023 19:01
@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

Merging #225 (5bbf4ca) into main (a5729b6) will increase coverage by 0.17%.
Report is 2 commits behind head on main.
The diff coverage is 82.35%.

@@            Coverage Diff             @@
##             main     #225      +/-   ##
==========================================
+ Coverage   90.81%   90.99%   +0.17%     
==========================================
  Files          67       67              
  Lines       12838    12645     -193     
  Branches     1268     1260       -8     
==========================================
- Hits        11659    11506     -153     
+ Misses       1156     1116      -40     
  Partials       23       23              
Components Coverage Δ
api 94.32% <100.00%> (ø)
common 95.00% <ø> (ø)
credentials 92.77% <ø> (ø)
crypto 100.00% <ø> (ø)
dids 92.16% <ø> (ø)
agent 88.16% <80.32%> (+0.35%) ⬆️
identity-agent 59.05% <ø> (ø)
proxy-agent 58.59% <ø> (ø)
user-agent 57.36% <ø> (ø)

@diehuxx diehuxx changed the title Agent dwn sdk js v0.2.2 Upgrade to dwn sdk js v0.2.2 Sep 28, 2023
@diehuxx diehuxx changed the title Upgrade to dwn sdk js v0.2.2 Upgrade to dwn sdk js v0.2.3 Sep 28, 2023
Copy link
Contributor

@thehenrytsai thehenrytsai left a comment

Choose a reason for hiding this comment

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

Looks good with massive amount of code removed!

Copy link
Contributor

@frankhinek frankhinek left a comment

Choose a reason for hiding this comment

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

Huge thanks for removing the singing patches 🥇

@diehuxx diehuxx merged commit 0fd3b3b into main Oct 3, 2023
21 of 23 checks passed
diehuxx pushed a commit to amika-sq/web5-js that referenced this pull request Oct 30, 2023
* main:
  Did ion: Set assertion method (decentralized-identity#245)
  Update README to reflect dwn-sdk-js 0.2.3 (decentralized-identity#237)
  changed the typo (decentralized-identity#234)
  Upgrade to dwn sdk js v0.2.3 (decentralized-identity#225)
@frankhinek frankhinek deleted the agent-dwn-sdk-js-v0.2.2 branch November 3, 2023 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants