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

build: release v6.108.0 #7067

Merged
merged 3 commits into from
Feb 6, 2024
Merged

build: release v6.108.0 #7067

merged 3 commits into from
Feb 6, 2024

Conversation

wanlingt
Copy link
Contributor

@wanlingt wanlingt commented Feb 6, 2024

New

  • feat(payment): webhook with charge information #7058
  • build: merge release v6.107.0 into develop #7064
  • build: release v6.107.0 #7063

Dependencies

Dev-Dependencies

Tests

feat(payment): webhook with charge information #7058

@wanlingt
Regression on non-payment forms

  • Create a form without payment fields
  • Setup webhook
  • Observe that submission is successful
  • Observe that webhook is sent with fields paymentContent: {} + encryptedContent
  • Observe that webhook retry is sent with fields paymentContent: {} + encryptedContent

paymentContent webhook data is sent for payment forms

  • Create a form with payment fields
  • Setup webhook
  • Observe that payment submission is successful
  • Observe that webhook is sent with correct fields paymentContent: {...} + encryptedContent
  • Observe that webhook retry is sent with correct fields paymentContent: {...} + encryptedContent

KenLSM and others added 3 commits February 5, 2024 13:21
build: merge release v6.107.0 into develop
* feat: populate payment fields

* fix: incorrect populate params

* fix: update test cases to include paymentContent

* feat: update paymentFields to be object mapped instead of array

* fix: fully resolve receipt download url

* fix: update import from absolute to relative

* test: add webhook tests for paymentContents

* feat: set empty webhooks for fixed payment

* refactor: add contextual comments for payment webhook event object

* fix: typing on paymetnContent
Copy link
Contributor

@kathleenkhy kathleenkhy left a comment

Choose a reason for hiding this comment

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

lgtm!

@wanlingt wanlingt merged commit 963ea16 into release-al2 Feb 6, 2024
55 of 57 checks passed
@wanlingt wanlingt deleted the release_v6.108.0 branch February 6, 2024 03:00
@tshuli tshuli mentioned this pull request Feb 14, 2024
9 tasks
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.

None yet

4 participants