Skip to content

fix: added goalCode, outOfBandRecordId, credentialRequestThId in credential controller#235

Merged
pranalidhanavade merged 5 commits intodevelopfrom
fix/inconsistent-webhook-responses
Feb 21, 2025
Merged

fix: added goalCode, outOfBandRecordId, credentialRequestThId in credential controller#235
pranalidhanavade merged 5 commits intodevelopfrom
fix/inconsistent-webhook-responses

Conversation

@pranalidhanavade
Copy link
Copy Markdown
Contributor

@pranalidhanavade pranalidhanavade commented Feb 20, 2025

What?

  • Added goal code to outofBandRecord in the credential controller.

Why?

  • To ensure response consistency in credential controller.

…ential controller

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>
@pranalidhanavade pranalidhanavade self-assigned this Feb 20, 2025
@pranalidhanavade pranalidhanavade changed the title fix: added goalCode, outOfBandRecordId, credentialRequestThId in cred… fix: added goalCode, outOfBandRecordId, credentialRequestThId in credential controller Feb 20, 2025
Comment thread src/controllers/credentials/CredentialController.ts
Comment thread src/events/CredentialEvents.ts Outdated
…controller

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>
Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>
Copy link
Copy Markdown
Member

@sairanjit sairanjit left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>
Copy link
Copy Markdown
Contributor

@tipusinghaw tipusinghaw left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>
@sonarqubecloud
Copy link
Copy Markdown

@pranalidhanavade pranalidhanavade merged commit 9076c57 into develop Feb 21, 2025
pranalidhanavade added a commit that referenced this pull request Mar 6, 2025
…ential controller (#235)

* fix: added goalCode, outOfBandRecordId, credentialRequestThId in credential controller

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added conditions for connectionId and outOfBandId in credential controller

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added optional chaining for connectionId record

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added extra check contextCorrelationId in credential events

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: removed extra check for contextCorrelationId

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

---------

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>
KambleSahil3 pushed a commit that referenced this pull request Apr 30, 2025
…ential controller (#235)

* fix: added goalCode, outOfBandRecordId, credentialRequestThId in credential controller

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added conditions for connectionId and outOfBandId in credential controller

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added optional chaining for connectionId record

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added extra check contextCorrelationId in credential events

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: removed extra check for contextCorrelationId

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

---------

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>
KambleSahil3 pushed a commit that referenced this pull request Apr 30, 2025
…ential controller (#235)

* fix: added goalCode, outOfBandRecordId, credentialRequestThId in credential controller

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added conditions for connectionId and outOfBandId in credential controller

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added optional chaining for connectionId record

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: added extra check contextCorrelationId in credential events

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

* fix: removed extra check for contextCorrelationId

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>

---------

Signed-off-by: pranalidhanavade <pranali.dhanavade@ayanworks.com>
Signed-off-by: KambleSahil3 <sahil.kamble@ayanworks.com>
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