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.

Sign up for GitHub

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

Jump to bottom

docs: using postgres storage #824

Merged
merged 40 commits into from Jun 3, 2022
Merged

docs: using postgres storage #824

merged 40 commits into from Jun 3, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jun 1, 2022 •
edited by ghost
Loading

Added documentation for usage and setup of Postgres plugin setup

  • Linux
  • Mac
  • Windows (Not able to build Postgres plugin on windows yet, getting an error regarding libsodium-sys 0.0.16 )

#758 #771

@ghost ghost self-requested a review as a code owner June 1, 2022 13:37
@ghost ghost mentioned this pull request Jun 1, 2022
Closed
sairanjit and others added 28 commits June 1, 2022 19:13
@sairanjit
docs: update configuration options
ae157f9 
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@sairanjit
docs: add linux documentation for postgres plugin
1c12ec5 
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@sairanjit
docs: add macos documentation for postgres plugin
540701f 
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@sairanjit
docs: add getting started documentation for postgres plugin
fd4e5df 
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
fix(node): allow to import node package without postgres (#757)
7444f06 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
fix: mediation record checks for pickup v2 (#736)
0985549 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@NB-MikeRichardson @sairanjit
fix: delete credentials (#766)
002af8c 
Signed-off-by: Mike Richardson <mike.richardson@northernblock.io>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@jakubkoci @sairanjit
feat: add out-of-band and did exchange (#717)
0536008 
Signed-off-by: Jakub Koci <jakub.koci@gmail.com>
Co-authored-by: Timo Glastra <timo@animo.id>

BREAKING CHANGE: the connections module has been extended with an out of band module and support for the DID Exchange protocol. Some methods have been moved to the out of band module, see [Migrating from AFJ 0.1.0 to 0.2.x](https://github.com/hyperledger/aries-framework-javascript/blob/main/docs/migration/0.1-to-0.2.md) for detailed migration instructions.

Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
feat: support handling messages with different minor version (#714)
1778788 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@sairanjit
docs: update dev readme
a849fac 
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@marcinkiewiczblazej @sairanjit
fix: relax validation of thread id in revocation notification (#768)
03eaef2 
Signed-off-by: Blazej Marcinkiewicz <marcinkiewicz@procivis.ch>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@MosCD3 @berendsliedrecht
@TimoGlastra @sairanjit
feat: ability to add generic records (#702)
1f1820d 
feat: extension module creation (#688)

Co-authored-by: Berend Sliedrecht <berend@animo.id>
Co-authored-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@NB-MikeRichardson @sairanjit
fix: propose payload attachment in in snake_case JSON format (#775)
78bb190 
Signed-off-by: Mike Richardson <mike.richardson@northernblock.io>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@NB-MikeRichardson @sairanjit
fix: delete credentials (#770)
d5cf155 
Signed-off-by: Mike Richardson <mike.richardson@northernblock.io>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
test: use event listener instead of while loop (#778)
1bfeccd 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
fix: add oob state and role check (#777)
bc4cf8f 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@karimStekelenburg @sairanjit
fix: add BBS context to DidDoc (#789)
5516806 
Signed-off-by: Karim <karim@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
feat: 0.2.0 migration script for connections (#773)
4a0742b 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
docs: fix demo for oob (#779)
a6f862d 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@NB-MikeRichardson @sairanjit
fix: miscellaneous issue credential v2 fixes (#769)
2932e96 
Signed-off-by: Mike Richardson <mike.richardson@northernblock.io>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
fix: remove unqualified did from out of band record (#782)
fcab54e 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
refactor: do not add ~service in createOOBOffer method (#772)
7d58dce 
Signed-off-by: Timo Glastra <timo@animo.id>

BREAKING CHANGE: with the addition of the out of band module `credentials.createOutOfBandOffer` is renamed to `credentials.createOffer` and no longer adds the `~service` decorator to the message. You need to call `oob.createLegacyConnectionlessInvitation` afterwards to use it for AIP-1 style connectionless exchanges. See [Migrating from AFJ 0.1.0 to 0.2.x](https://github.com/hyperledger/aries-framework-javascript/blob/main/docs/migration/0.1-to-0.2.md) for detailed migration instructions.

Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@berendsliedrecht @sairanjit
fix(core): expose CredentialPreviewAttribute (#796)
23d5fee 
Signed-off-by: Berend Sliedrecht <berend@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@Amit-Padmani @sairanjit
fix: extract indy did from peer did in indy credential request (#790)
2dcebdb 
Signed-off-by: Amit-Padmani <amit.padmani@ontario.ca>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
ci: run ci on -pre branches (#799)
6dc654b 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@KolbyRKunz @sairanjit
fix(routing): sending of trustping in pickup v2 (#787)
0a0eb8c 
Signed-off-by: KolbyRKunz <KolbyKunz@yahoo.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@NB-MikeRichardson @sairanjit
fix(credentials): add missing issue credential v1 proposal attributes (…
9ba16a5 
…#798)

Signed-off-by: Mike Richardson <mike.richardson@ontario.ca>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@berendsliedrecht @sairanjit
refactor(credentials): minor refactor of CredentialMessageBuilder (#797)
87fd82f 
Signed-off-by: Berend Sliedrecht <berend@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
NB-MikeRichardson and others added 10 commits June 1, 2022 19:13
@NB-MikeRichardson @sairanjit
refactor(credentials): retrieve connection in credential module (#776)
44b0151 
Signed-off-by: Mike Richardson <mike.richardson@ontario.ca>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@NB-MikeRichardson @sairanjit
refactor: delete credentials by default when deleting exchange (#767)
c7bf0ba 
Signed-off-by: Mike Richardson <mike.richardson@ontario.ca>

BREAKING CHANGE: the credentials associated with a credential exchange record are now deleted by default when deleting a credential exchange record. If you only want to delete the credential exchange record and not the associated credentials, you can pass the deleteAssociatedCredentials to the deleteById method:

```ts
await agent.credentials.deleteById('credentialExchangeId', {
	deleteAssociatedCredentials: false
})
```

Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
fix(routing): also use pickup strategy from config (#808)
af2c614 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@berendsliedrecht @TimoGlastra @sairanjit
fix(core): allow JSON as input for indy attributes (#813)
34f3053 
Signed-off-by: Berend Sliedrecht <berend@animo.id>

Co-authored-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@TimoGlastra @sairanjit
fix(oob): check service is string instance (#814)
d8ea5c3 
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@NB-MikeRichardson @sairanjit
fix(credentials): default for credentials in exchange record (#816)
cf0e789 
Signed-off-by: Mike Richardson <mike.richardson@ontario.ca>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@NB-MikeRichardson @sairanjit
refactor(credentials): separate offer interfaces (#819)
8554e7d 
Signed-off-by: Mike Richardson <mike.richardson@ontario.ca>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@MosCD3 @sairanjit
docs: add ontario copyright license (#815)
b1392f5 
Signed-off-by: Mostafa Gamal <46829557+MosCD3@users.noreply.github.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@sairanjit
docs: update the docs for mac
a08d1a5 
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit.tummalapalli@ayanworks.com>
@sairanjit
Merge branch 'main' of https://github.com/sairanjitAW/aries-framework…
a369285 
…-javascript into docs/using-postgres-storage
@codecov-commenter
Copy link

codecov-commenter commented Jun 1, 2022 •
edited
Loading

Codecov Report

Merging #824 (269cf66) into main (8dd7f80) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #824   +/-   ##
=======================================
  Coverage   87.69%   87.69%           
=======================================
  Files         438      438           
  Lines       10820    10820           
  Branches     1891     1891           
=======================================
  Hits         9489     9489           
  Misses       1269     1269           
  Partials       62       62

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8dd7f80...269cf66. Read the comment docs.

TimoGlastra
TimoGlastra reviewed Jun 1, 2022
View reviewed changes
Copy link
Contributor

@TimoGlastra TimoGlastra left a comment

Choose a reason for hiding this comment

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

@blu3beri can you review this PR and see how we can integrate this with the new documentation site?

@berendsliedrecht
Copy link
Contributor

I think it good to merge it. I will most like change some minor parts when integrating it into the new documentation website but for now this will suffice. Thanks for the PR @sairanjitAW

@berendsliedrecht
Copy link
Contributor

@sairanjitAW do you think that Windows will be done soon? I think merging it after that might be the best.

@ghost
Copy link
Author

ghost commented Jun 2, 2022

@blu3beri I'm working on it, but it will take some time.
Still stuck in building libsodium-sys-0.0.16 on windows.

@TimoGlastra
Copy link
Contributor

In that case maybe we can already merge it? As postgres support has already been merged this seems quite useful

@berendsliedrecht berendsliedrecht enabled auto-merge (squash) June 3, 2022 10:31
@berendsliedrecht berendsliedrecht merged commit a717a58 into openwallet-foundation:main Jun 3, 2022
@ghost ghost deleted the docs/using-postgres-storage branch June 3, 2022 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.