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

feat: expose findAllByQuery method in modules and services #1044

Merged
merged 14 commits into from
Oct 6, 2022
Merged

feat: expose findAllByQuery method in modules and services #1044

merged 14 commits into from
Oct 6, 2022

Conversation

jimezesinachi
Copy link
Contributor

…ecessary tests for the switch

Signed-off-by: Jim Ezesinachi ezesinachijim@gmail.com

@jimezesinachi
exposed findByQuery method in module services and added some of the n…
ca2c18e 
…ecessary tests for the switch

Signed-off-by: Jim Ezesinachi <ezesinachijim@gmail.com>
@jimezesinachi jimezesinachi requested a review from a team as a code owner October 5, 2022 13:26
genaris
genaris requested changes Oct 5, 2022
View reviewed changes
Copy link
Contributor

@genaris genaris left a comment

Choose a reason for hiding this comment

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

Thanks @rapaktech ! I think this will be very useful. I left some comments in the code but I would like to add some general suggestions that maybe others can comment on as well:

  • I would prefer to use the naming 'findAllByQuery' to emphasize that multiple records could be retrieved. This is the same naming as used in other modules such as basic-messages and generic-records
  • As I think this is related to Expose findByQuery methods in public API of all modules #1022, it would be nice to expose these methods in the corresponding module classes. If I'm not wrong, you can even modify the parameters of findAllByQuery of BasicMessagesModule and GenericRecordsModule without introducing a breaking change, since Query<T extends BaseRecord> contains Partial<RecordTags<T extends BaseRecord>>

packages/core/src/modules/credentials/services/CredentialService.ts Outdated Show resolved Hide resolved
jimezesinachi and others added 2 commits October 5, 2022 16:58
@jimezesinachi @genaris
Update packages/core/src/modules/action-menu/services/ActionMenuServi…
5ce9773 
…ce.ts

Co-authored-by: Ariel Gentile <gentilester@gmail.com>
Signed-off-by: Jim Ezesinachi <ezesinachijim@gmail.com>
@jimezesinachi
renamed exposed methods from to method
79b7858 
Signed-off-by: Jim Ezesinachi <jim@animo.id>
@jimezesinachi
Copy link
Contributor Author

I'm sill working on the changes.. I'll re-request a review when I'm done

@jimezesinachi
exposed more findByQuery methods and fixed failing import
f035d0c 
Signed-off-by: Jim Ezesinachi <jim@animo.id>
@jimezesinachi
fix: fix linting issue
811ad6a 
Signed-off-by: Jim Ezesinachi <jim@animo.id>
@jimezesinachi
fix: fix linting issue
430afb0 
Signed-off-by: Jim Ezesinachi <jim@animo.id>
@jimezesinachi jimezesinachi changed the title exposed findByQuery method in module services and added some of the n… feat: exposed findByQuery method in module services and added some of the necessary tests for the switch Oct 5, 2022
jimezesinachi and others added 3 commits October 6, 2022 09:26
@jimezesinachi @TimoGlastra
Update packages/core/src/modules/routing/services/MediationRecipientS…
f497f1c 
…ervice.ts

Co-authored-by: Timo Glastra <timo@animo.id>
Signed-off-by: Jim Ezesinachi <ezesinachijim@gmail.com>
@jimezesinachi
fix: fix QuestionAnswer service params and renamed mediator service m…
f1a13ab 
…ethod

Signed-off-by: Jim Ezesinachi <jim@animo.id>
@jimezesinachi
feat: swiched BasicMessageRecord & GenericRecord from Partial to Quer…
01a7b45 
…y type

Signed-off-by: Jim Ezesinachi <jim@animo.id>
@codecov-commenter
Copy link

codecov-commenter commented Oct 6, 2022 •
edited
Loading

Codecov Report

Merging #1044 (091dd57) into main (0d14a71) will decrease coverage by 0.15%.
The diff coverage is 30.00%.

@@            Coverage Diff             @@
##             main    #1044      +/-   ##
==========================================
- Coverage   88.74%   88.58%   -0.16%     
==========================================
  Files         522      522              
  Lines       12156    12180      +24     
  Branches     1913     1913              
==========================================
+ Hits        10788    10790       +2     
- Misses       1364     1386      +22     
  Partials        4        4
Impacted Files Coverage Δ
.../modules/action-menu/services/ActionMenuService.ts 96.68% <0.00%> (-1.30%) ⬇️
.../src/modules/basic-messages/BasicMessagesModule.ts 93.33% <0.00%> (ø)
...les/basic-messages/services/BasicMessageService.ts 92.59% <0.00%> (ø)
.../core/src/modules/connections/ConnectionsModule.ts 70.06% <0.00%> (-0.97%) ⬇️
.../core/src/modules/credentials/CredentialsModule.ts 87.85% <0.00%> (-0.83%) ⬇️
packages/core/src/modules/oob/OutOfBandModule.ts 84.58% <0.00%> (-0.59%) ⬇️
packages/core/src/modules/proofs/ProofsModule.ts 78.12% <0.00%> (-0.99%) ⬇️
...s/core/src/modules/proofs/services/ProofService.ts 85.67% <0.00%> (-0.50%) ⬇️
...rc/modules/question-answer/QuestionAnswerModule.ts 91.48% <0.00%> (-4.07%) ⬇️
.../question-answer/services/QuestionAnswerService.ts 92.10% <0.00%> (ø)
... and 10 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@genaris genaris linked an issue Oct 6, 2022 that may be closed by this pull request
Expose findByQuery methods in public API of all modules #1022
Closed
genaris
genaris requested changes Oct 6, 2022
View reviewed changes
Copy link
Contributor

@genaris genaris left a comment

Choose a reason for hiding this comment

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

Looks pretty good and I actually approved it but noticed some remaining issues afterwards.

packages/core/src/storage/StorageService.ts Show resolved Hide resolved
samples/extension-module/dummy/DummyApi.ts Outdated Show resolved Hide resolved
samples/extension-module/dummy/services/DummyService.ts Outdated Show resolved Hide resolved
@genaris genaris changed the title feat: exposed findByQuery method in module services and added some of the necessary tests for the switch feat: expose findAllByQuery method in modules and services Oct 6, 2022
jimezesinachi and others added 5 commits October 6, 2022 20:24
@jimezesinachi @genaris
Update samples/extension-module/dummy/services/DummyService.ts
f6048d9 
Co-authored-by: Ariel Gentile <gentilester@gmail.com>
Signed-off-by: Jim Ezesinachi <ezesinachijim@gmail.com>
@jimezesinachi
feat: exported Query type from packages/core/src/index and replaced Q…
f71e5b5 
…uery import in DummyApi and DummyService with the new Query export

Signed-off-by: Jim Ezesinachi <jim@animo.id>
@jimezesinachi
feat: exported Query type from packages/core/src/index and replaced Q…
091dd57 
…uery import in DummyApi and DummyService with the new Query export

Signed-off-by: Jim Ezesinachi <jim@animo.id>
@genaris
chore: fix import and remove eslint ignore line
1aad7c3 
Signed-off-by: Ariel Gentile <gentilester@gmail.com>
@genaris
Merge remote-tracking branch 'upstream/main' into feat/expose-query-m…
ecd1057 
…ethods
@genaris genaris enabled auto-merge (squash) October 6, 2022 23:57
@genaris genaris disabled auto-merge October 6, 2022 23:57
@genaris genaris enabled auto-merge (squash) October 6, 2022 23:58
@genaris genaris merged commit 9dd95e8 into openwallet-foundation:main Oct 6, 2022
TimoGlastra added a commit that referenced this pull request Oct 11, 2022
@genaris @Iskander508
@conanoc @github-actions @niall-shaw @jakubkoci @TimoGlastra @garretaserra @sairanjit @KolbyRKunz @jimezesinachi @an-uhryn
chore: merge branch 'main' into 0.3.0-pre (#1030)
34658b0 
* feat: OOB public did (#930)

Signed-off-by: Pavel Zarecky <zarecky@procivis.ch>

* feat(routing): manual mediator pickup lifecycle management (#989)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* docs(demo): faber creates invitation (#995)

Signed-off-by: conanoc <conanoc@gmail.com>

* chore(release): v0.2.3 (#999)

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix(question-answer): question answer protocol state/role check (#1001)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* feat: Action Menu protocol (Aries RFC 0509) implementation (#974)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* fix(ledger): remove poolConnected on pool close (#1011)

Signed-off-by: Niall Shaw <niall.shaw@absa.africa>

* fix(ledger): check taa version instad of aml version (#1013)

Signed-off-by: Jakub Koci <jakub.koci@gmail.com>

* chore: add @janrtvld to maintainers (#1016)

Signed-off-by: Timo Glastra <timo@animo.id>

* feat(routing): add settings to control back off strategy on mediator reconnection (#1017)

Signed-off-by: Sergi Garreta <sergi.garreta@entrust.com>

* fix: avoid crash when an unexpected message arrives (#1019)

Signed-off-by: Pavel Zarecky <zarecky@procivis.ch>

* chore(release): v0.2.4 (#1024)

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* style: fix some lint errors

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* feat: use did:key flag (#1029)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* ci: set default rust version (#1036)

Signed-off-by: Sai Ranjit Tummalapalli <sairanjit5@gmail.com>

* fix(oob): allow encoding in content type header (#1037)

Signed-off-by: Timo Glastra <timo@animo.id>

* feat: connection type (#994)

Signed-off-by: KolbyRKunz <KolbyKunz@yahoo.com>

* chore(module-tenants): match package versions

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* feat: improve sending error handling (#1045)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* feat: expose findAllByQuery method in modules and services (#1044)

Signed-off-by: Jim Ezesinachi <jim@animo.id>

* feat: possibility to set masterSecretId inside of WalletConfig (#1043)

Signed-off-by: Andrii Uhryn <an.ugryn@gmail.com>

* fix(oob): set connection alias when creating invitation (#1047)

Signed-off-by: Jakub Koci <jakub.koci@gmail.com>

* build: fix missing parameter

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

Signed-off-by: Pavel Zarecky <zarecky@procivis.ch>
Signed-off-by: Ariel Gentile <gentilester@gmail.com>
Signed-off-by: conanoc <conanoc@gmail.com>
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Niall Shaw <niall.shaw@absa.africa>
Signed-off-by: Jakub Koci <jakub.koci@gmail.com>
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sergi Garreta <sergi.garreta@entrust.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit5@gmail.com>
Signed-off-by: KolbyRKunz <KolbyKunz@yahoo.com>
Signed-off-by: Jim Ezesinachi <jim@animo.id>
Signed-off-by: Andrii Uhryn <an.ugryn@gmail.com>
Co-authored-by: Iskander508 <pavel.zarecky@seznam.cz>
Co-authored-by: conanoc <conanoc@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Niall Shaw <100220424+niallshaw-absa@users.noreply.github.com>
Co-authored-by: jakubkoci <jakub.koci@gmail.com>
Co-authored-by: Timo Glastra <timo@animo.id>
Co-authored-by: Sergi Garreta Serra <garretaserra@gmail.com>
Co-authored-by: Sai Ranjit Tummalapalli <34263716+sairanjit@users.noreply.github.com>
Co-authored-by: KolbyRKunz <KolbyKunz@yahoo.com>
Co-authored-by: Jim Ezesinachi <ezesinachijim@gmail.com>
Co-authored-by: an-uhryn <55444541+an-uhryn@users.noreply.github.com>
genaris added a commit to 2060-io/aries-framework-javascript that referenced this pull request Oct 13, 2022
@genaris @Iskander508
@conanoc @github-actions @niall-shaw @jakubkoci @TimoGlastra @garretaserra @sairanjit @KolbyRKunz @jimezesinachi @an-uhryn
chore: merge branch 'main' into 0.3.0-pre (openwallet-foundation#1030)
edd81bc 
* feat: OOB public did (openwallet-foundation#930)

Signed-off-by: Pavel Zarecky <zarecky@procivis.ch>

* feat(routing): manual mediator pickup lifecycle management (openwallet-foundation#989)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* docs(demo): faber creates invitation (openwallet-foundation#995)

Signed-off-by: conanoc <conanoc@gmail.com>

* chore(release): v0.2.3 (openwallet-foundation#999)

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix(question-answer): question answer protocol state/role check (openwallet-foundation#1001)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* feat: Action Menu protocol (Aries RFC 0509) implementation (openwallet-foundation#974)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* fix(ledger): remove poolConnected on pool close (openwallet-foundation#1011)

Signed-off-by: Niall Shaw <niall.shaw@absa.africa>

* fix(ledger): check taa version instad of aml version (openwallet-foundation#1013)

Signed-off-by: Jakub Koci <jakub.koci@gmail.com>

* chore: add @janrtvld to maintainers (openwallet-foundation#1016)

Signed-off-by: Timo Glastra <timo@animo.id>

* feat(routing): add settings to control back off strategy on mediator reconnection (openwallet-foundation#1017)

Signed-off-by: Sergi Garreta <sergi.garreta@entrust.com>

* fix: avoid crash when an unexpected message arrives (openwallet-foundation#1019)

Signed-off-by: Pavel Zarecky <zarecky@procivis.ch>

* chore(release): v0.2.4 (openwallet-foundation#1024)

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* style: fix some lint errors

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* feat: use did:key flag (openwallet-foundation#1029)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* ci: set default rust version (openwallet-foundation#1036)

Signed-off-by: Sai Ranjit Tummalapalli <sairanjit5@gmail.com>

* fix(oob): allow encoding in content type header (openwallet-foundation#1037)

Signed-off-by: Timo Glastra <timo@animo.id>

* feat: connection type (openwallet-foundation#994)

Signed-off-by: KolbyRKunz <KolbyKunz@yahoo.com>

* chore(module-tenants): match package versions

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* feat: improve sending error handling (openwallet-foundation#1045)

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

* feat: expose findAllByQuery method in modules and services (openwallet-foundation#1044)

Signed-off-by: Jim Ezesinachi <jim@animo.id>

* feat: possibility to set masterSecretId inside of WalletConfig (openwallet-foundation#1043)

Signed-off-by: Andrii Uhryn <an.ugryn@gmail.com>

* fix(oob): set connection alias when creating invitation (openwallet-foundation#1047)

Signed-off-by: Jakub Koci <jakub.koci@gmail.com>

* build: fix missing parameter

Signed-off-by: Ariel Gentile <gentilester@gmail.com>

Signed-off-by: Pavel Zarecky <zarecky@procivis.ch>
Signed-off-by: Ariel Gentile <gentilester@gmail.com>
Signed-off-by: conanoc <conanoc@gmail.com>
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Niall Shaw <niall.shaw@absa.africa>
Signed-off-by: Jakub Koci <jakub.koci@gmail.com>
Signed-off-by: Timo Glastra <timo@animo.id>
Signed-off-by: Sergi Garreta <sergi.garreta@entrust.com>
Signed-off-by: Sai Ranjit Tummalapalli <sairanjit5@gmail.com>
Signed-off-by: KolbyRKunz <KolbyKunz@yahoo.com>
Signed-off-by: Jim Ezesinachi <jim@animo.id>
Signed-off-by: Andrii Uhryn <an.ugryn@gmail.com>
Co-authored-by: Iskander508 <pavel.zarecky@seznam.cz>
Co-authored-by: conanoc <conanoc@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Niall Shaw <100220424+niallshaw-absa@users.noreply.github.com>
Co-authored-by: jakubkoci <jakub.koci@gmail.com>
Co-authored-by: Timo Glastra <timo@animo.id>
Co-authored-by: Sergi Garreta Serra <garretaserra@gmail.com>
Co-authored-by: Sai Ranjit Tummalapalli <34263716+sairanjit@users.noreply.github.com>
Co-authored-by: KolbyRKunz <KolbyKunz@yahoo.com>
Co-authored-by: Jim Ezesinachi <ezesinachijim@gmail.com>
Co-authored-by: an-uhryn <55444541+an-uhryn@users.noreply.github.com>
@jimezesinachi jimezesinachi deleted the feat/expose-query-methods branch November 16, 2022 10:45
@berendsliedrecht berendsliedrecht mentioned this pull request Jan 18, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TimoGlastra TimoGlastra TimoGlastra left review comments

@genaris genaris genaris approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Expose findByQuery methods in public API of all modules
4 participants
@jimezesinachi @codecov-commenter @genaris @TimoGlastra