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

fix: info element added & logs addition #471

Merged
merged 9 commits into from
Jul 4, 2024
Merged

Conversation

PritishBudhiraja
Copy link
Collaborator

@PritishBudhiraja PritishBudhiraja commented Jul 3, 2024

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Adding of Info Element and Logs

Screenshot 2024-07-04 at 12 35 52 PM

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

@PritishBudhiraja PritishBudhiraja changed the title fix: info element added fix: info element added & logs addition Jul 3, 2024
@PritishBudhiraja PritishBudhiraja self-assigned this Jul 4, 2024
@PritishBudhiraja PritishBudhiraja added the Ready for Review PR with label Ready for Review should only be reviewed. label Jul 4, 2024
@PritishBudhiraja PritishBudhiraja linked an issue Jul 4, 2024 that may be closed by this pull request
@vsrivatsa-edinburgh
Copy link
Member

vsrivatsa-edinburgh commented Jul 4, 2024

Include the newly added API events in calculateLatencyHook
PAYMENT_METHODS_AUTH_EXCHANGE_CALL
PAYMENT_METHODS_AUTH_LINK_CALL

Copy link
Member

@vsrivatsa-edinburgh vsrivatsa-edinburgh left a comment

Choose a reason for hiding this comment

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

Changes needed

src/Payments/AddBankDetails.res Outdated Show resolved Hide resolved
src/Payments/PlaidSDKIframe.res Outdated Show resolved Hide resolved
src/Payments/PlaidSDKIframe.res Outdated Show resolved Hide resolved
src/Utilities/PaymentHelpers.res Show resolved Hide resolved
src/Utilities/PaymentHelpers.res Outdated Show resolved Hide resolved
src/Utilities/PaymentHelpers.res Show resolved Hide resolved
src/Utilities/PaymentHelpers.res Outdated Show resolved Hide resolved
src/Utilities/PaymentHelpers.res Outdated Show resolved Hide resolved
src/Utilities/PaymentHelpers.res Outdated Show resolved Hide resolved
src/Utilities/PaymentHelpers.res Outdated Show resolved Hide resolved
@PritishBudhiraja PritishBudhiraja merged commit 53b3961 into main Jul 4, 2024
3 checks passed
@PritishBudhiraja PritishBudhiraja deleted the plaid-info-logs branch July 4, 2024 07:30
@github-actions github-actions bot added the Closed Label will be automatically added when the PR will get merged to main label Jul 4, 2024
akash-c-k pushed a commit that referenced this pull request Jul 4, 2024
## [0.72.2](v0.72.1...v0.72.2) (2024-07-04)

### Bug Fixes

* info element added & logs addition ([#471](#471)) ([53b3961](53b3961))
@PritishBudhiraja PritishBudhiraja removed the Ready for Review PR with label Ready for Review should only be reviewed. label Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Label will be automatically added when the PR will get merged to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding logs and Info element for Plaid SDK integration
2 participants