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

feat: use published bifold #1647

Merged
merged 20 commits into from
Dec 6, 2023
Merged

feat: use published bifold #1647

merged 20 commits into from
Dec 6, 2023

Conversation

jleach
Copy link
Member

@jleach jleach commented Dec 2, 2023

Adapt BC Wallet to use the @hyperledger/aries-bifold-core packages published to NPM.

cvarjao and others added 5 commits December 1, 2023 12:38
Signed-off-by: Clécio Varjão <[email protected]>
Signed-off-by: Clécio Varjão <[email protected]>
Signed-off-by: Clécio Varjão <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
@jleach jleach self-assigned this Dec 2, 2023
@jleach jleach requested a review from wadeking98 December 2, 2023 00:18
Signed-off-by: Jason C. Leach <[email protected]>
@jleach jleach marked this pull request as draft December 2, 2023 00:34
@@ -59,7 +56,6 @@
"@aries-framework/react-hooks@^0.4.2": "patch:@aries-framework/react-hooks@npm:0.4.2#./.yarn/patches/@aries-framework-react-hooks-npm-0.4.2-84b7eb8764.patch",
"@aries-framework/[email protected]": "patch:@aries-framework/react-native@npm:0.3.3#./.yarn/patches/@aries-framework-react-native-npm-0.3.3-bb75ece22d.patch",
"@aries-framework/anoncreds@^0.4.0": "patch:@aries-framework/anoncreds@npm%3A0.4.0#./.yarn/patches/@aries-framework-anoncreds-npm-0.4.0-4d3b4e769d.patch",
"@aries-framework/[email protected]": "patch:@aries-framework/anoncreds@npm%3A0.4.0#./.yarn/patches/@aries-framework-anoncreds-npm-0.4.0-4d3b4e769d.patch",
"@aries-framework/indy-vdr@^0.4.0": "patch:@aries-framework/indy-vdr@npm%3A0.4.0#./.yarn/patches/@aries-framework-indy-vdr-npm-0.4.0-b01067562c.patch"
Copy link
Member

Choose a reason for hiding this comment

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

is this being removed by accident?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure what happened here. I still see @aries-framework/[email protected] and added VDR back.

Copy link
Contributor

Choose a reason for hiding this comment

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

there were two anoncreds patches here, I believe one was the patch to the revocation interval that jason made, The other was a speed improvement from me when looking up the revocation registry for multiple credentials. Have the patches been combined or was one accidentally deleted?

Copy link
Contributor

Choose a reason for hiding this comment

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

they appear to have been duplicates, looking at the hash

Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
@jleach jleach marked this pull request as ready for review December 4, 2023 19:57
@jleach jleach requested a review from cvarjao December 4, 2023 19:57
@jleach jleach marked this pull request as draft December 4, 2023 20:04
Signed-off-by: Jason C. Leach <[email protected]>
@jleach jleach marked this pull request as ready for review December 4, 2023 20:11
package.json Outdated Show resolved Hide resolved
jleach and others added 8 commits December 4, 2023 14:55
Signed-off-by: Jason C. Leach <[email protected]>
# delete all node_modules

find . -name 'node_modules' -type d -prune -exec rm -rf '{}' +

# reset lock file from main

git restore --source origin/main -- yarn.lock

# install dependencies

yarn install

Signed-off-by: Clécio Varjão <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Signed-off-by: Jason C. Leach <[email protected]>
Copy link

sonarqubecloud bot commented Dec 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 7 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jleach jleach enabled auto-merge (squash) December 6, 2023 02:27
@jleach jleach requested a review from bryce-mcmath December 6, 2023 02:27
@wadeking98
Copy link
Contributor

wadeking98 commented Dec 6, 2023

Other than my one comment LGTM

@jleach jleach merged commit 77d5fb1 into main Dec 6, 2023
9 checks passed
@jleach jleach deleted the use_published_bifold branch December 6, 2023 18:43
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.

4 participants