immobiliare / braze-php-sdk Goto Github PK

In my opinion, the readme is missing a section that talks about braze with a link to this service and also a section that highlights the strengths of the software.
WDYT?

Feature Request

Summary

SDK do not have users/merge implementation
https://www.braze.com/docs/api/endpoints/user_data/post_users_merge/

In the README we use still Instance. Please change with Region.

Change https://github.com/immobiliare/wsse-header-generator-php/workflows/CI/badge.svg with https://github.com/immobiliare/braze-php-sdk/workflows/CI/badge.svg

https://github.com/immobiliare/braze-php-sdk/blob/main/LICENSE#L3 ;)

Please add Issue and Pull Request Template in the .github dir!

Wait! In log of actions i can see this: https://github.com/immobiliare/braze-php-sdk/runs/4804082019?check_suite_focus=true#step:7:9

Originally posted by @JellyBellyDev in #11 (comment)

I added some keywords in the about section of GitHub repo, please can you add these also in composer.json?

Here https://github.com/immobiliare/braze-php-sdk/blob/main/composer.json#L14 we set >=7.2.0.
This approach in the world of php libraries is discouraged in favor of ^7.2 || ^8.0
WDYT to change?

Feature Request

Summary

Since dynamic properties have been deprecated in PHP 8.2, to set custom attributes in the UserAttributes object a setCustomAttribute method will be added in the same class. The custom attributes will be kept in an associative array, and support for their proper serialization in the request will be added.

Before implementation, suggestions for alternative solutions will be considered.