Production 2024-09-13_01 #2655
Merged
Production 2024-09-13_01 #2655
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
pavelbrm
commented
Sep 12, 2024
pavelbrm
commented
- feat: add radom checkout session integration for new orders #2627
- feat: use same issuer for monthly and annual products #2654
* feat: add radom checkout session integration for new orders * test: add no metadata test order item req * style: add new line to slice init in radom gateway * fix: set radom authorization header without bearer prefix * fix: rectify skus model tests after rebase * refactor: rename method createRadomSessID to createRadomSession * refactor: address comments * fix imports * update gateway to pointer * update radom checkout session request to pointer * rename order items to line items function * add radom product id getter on metadata * refactor: remove unused variable emptyCreateCheckoutSessionResp * refactor: remove unnecessary convert to UTC * refactor: sort imports * refactor: remove unused func mustDecimalFromString * feat: handle radom webhook events (#2640) * feat: add radom checkout session integration for new orders * fix: rectify skus model tests after rebase * refactor: address comments * fix imports * update gateway to pointer * update radom checkout session request to pointer * rename order items to line items function * add radom product id getter on metadata * feat: add radom checkout session integration for new orders * feat: handle radom webhook events * test: remove duplicate test and fix pointer after rebase * feat: set radom webhook verification key * refactor: use pointers for radom events * refactor: fix import order skus controller * refactor: simplify radom event processing checks * style: fix parse apostrophe * refactor: use pointer for radom subscription response * refactor: remove call to error when calling err on logging * refactor: use index directly on radom mdata * refactor: make radom action error naming less generic * refactor: use pointer for service in test * refactor: sort imports in radom.go * refactor: update radom empty payments error message * style: return from func call opposed to explicit return nil * refactor: use string in radom get subscription * refactor: add on conflict do nothing to order history insert statement * refactor: rename radom event to radom notification * test: remove duplicate test after rebase
* feat: use same issuer for monthly and annual product * fix: fix typo in parseVerifyCredOpaque * fix: use better field name and preserve error response * fix: use more precise language in log messages
clD11
approved these changes
Sep 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.