Jax DesMarais-Leder
Jax DesMarais-Leder
### Summary of changes - Convert BTAPIHTTP and BTGraphQLHTTP to Swift - There are some TODOs in here that will be addressed in follow up PRs - Delete BTHTTP, BTGraphQLHTTP,...
### Summary of changes - Move to `setListener()` pattern used in other modules - Remove unneeded else block ### Checklist - ~[ ] Added a changelog entry~ - ~[ ]...
### Summary of changes - Add Unit Tests for PayPal Messaging feature ### Checklist - ~[ ] Added a changelog entry~ - [x] Relevant test coverage ### Authors @jwarmkessel @jaxdesmarais
### Summary of changes - Add `setFallbackToWeb()` to `VenmoRequest` - If set to `true` customers will fallback to a web based Venmo flow if the Venmo app is not installed...
### Summary of changes - Send `link_type` in `batch_params` to PayPal's analytics service (FPTI) - this will help us determine if users are going through the universal link or deeplink...
### Summary of changes - Remove `HTTPURLResponse` from all HTTP requests - Update Tests #### Additional Details: This was only used in 1 place in the entire codebase when checking...
### Summary of changes - Add new `BTShopperInsightsError.invalidAuthorization` and check authorization type in `BTShopperInsightsClient` - This fixes a bug where you would see `SWIFT TASK CONTINUATION MISUSE: post(_:parameters:headers:httpType:) leaked its...
### Summary of changes - ### Checklist - ~[ ] Added a changelog entry~ ### Authors - @jaxdesmarais
### Summary of changes - Doing some other troubleshooting and noticed we were using `self` all over `ShopperInsightsViewController` when we didn't need to be. Cleaned it up to make the...
### Summary of changes - I noticed there were several places where our README was outdated so wanted to clean it up a bit ### Checklist - [x] Added a...