Skip to content
This repository has been archived by the owner on Jan 15, 2025. It is now read-only.

[#IOCIT-341] Implement GetAssertionHandler #26

Merged
merged 20 commits into from
Mar 6, 2023
Merged

[#IOCIT-341] Implement GetAssertionHandler #26

merged 20 commits into from
Mar 6, 2023

Conversation

gquadrati
Copy link
Contributor

@gquadrati gquadrati commented Mar 1, 2023 •
edited
Loading

List of Changes

Add GetAssertionHandler implementation

Motivation and Context

Enable LolliPoP Consumer to retrieve an assertion

How Has This Been Tested?

Unit tests, Integration tests

Types of changes

  • Chore (nothing changes by a user perspective)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@github-actions
Copy link

github-actions bot commented Mar 1, 2023

Jira Pull Request Link

This Pull Request refers to the following Jira issue IOCIT-341

@gquadrati gquadrati force-pushed the IOCIT-341--implement-getAssertion-handler branch from 29971c5 to dcfda88 Compare March 2, 2023 08:17
@gquadrati gquadrati marked this pull request as ready for review March 2, 2023 10:27
@gquadrati gquadrati requested a review from a team as a code owner March 2, 2023 10:27
@gquadrati gquadrati requested review from arcogabbo and Garma00 March 2, 2023 10:27
michaeldisaro
michaeldisaro reviewed Mar 2, 2023
View reviewed changes
utils/readers.ts Outdated Show resolved Hide resolved
michaeldisaro
michaeldisaro reviewed Mar 2, 2023
View reviewed changes
utils/readers.ts Outdated Show resolved Hide resolved
arcogabbo
arcogabbo reviewed Mar 3, 2023
View reviewed changes
GetAssertion/handler.ts Show resolved Hide resolved
GetAssertion/handler.ts Outdated Show resolved Hide resolved
__integrations__/__tests__/getAssertion.test.ts Outdated Show resolved Hide resolved
__integrations__/__tests__/getAssertion.test.ts Show resolved Hide resolved
__integrations__/environments/env.base Show resolved Hide resolved
__integrations__/__tests__/getAssertion.test.ts Outdated Show resolved Hide resolved
BurnedMarshal
BurnedMarshal reviewed Mar 6, 2023
View reviewed changes
GetAssertion/handler.ts Outdated Show resolved Hide resolved
GetAssertion/handler.ts Outdated Show resolved Hide resolved
GetAssertion/handler.ts Outdated Show resolved Hide resolved
GetAssertion/handler.ts Show resolved Hide resolved
utils/readers.ts Outdated Show resolved Hide resolved
__integrations__/__tests__/getAssertion.test.ts Outdated Show resolved Hide resolved
__integrations__/__tests__/getAssertion.test.ts Show resolved Hide resolved
__integrations__/__tests__/getAssertion.test.ts Outdated Show resolved Hide resolved
__integrations__/utils/client.ts Outdated Show resolved Hide resolved
__integrations__/__tests__/getAssertion.test.ts Show resolved Hide resolved
@BurnedMarshal BurnedMarshal merged commit 58b513b into master Mar 6, 2023
@BurnedMarshal BurnedMarshal deleted the IOCIT-341--implement-getAssertion-handler branch March 6, 2023 14:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@BurnedMarshal BurnedMarshal BurnedMarshal left review comments

@michaeldisaro michaeldisaro michaeldisaro approved these changes

@arcogabbo arcogabbo arcogabbo approved these changes

@Garma00 Garma00 Awaiting requested review from Garma00 Garma00 was automatically assigned from pagopa/io-backend-contributors

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

Successfully merging this pull request may close these issues.
5 participants
@gquadrati @BurnedMarshal @michaeldisaro @arcogabbo @AndreaPagoPa