* Bug fix - Allow adding custom headers with non-array value
* Remove whitespace
* Functional test for adding custom headers (either singular or multiple)
* Style fixes for functional test
* Renamed namespace from Resource\Api to Model
* Moved Resources\ApiResponse to Models namespace
Signed-off-by: Michal Hanajík <michal.hanajik@outlook.com>
* Fix of code style
* Add initial (empty) Routes PHP Unit Test file
We still need to provide the automated tests
* Add initial Routes file
* Describe API Methods
* Inherit method from TestCase - adding @inheritdoc annotation
* Add new DTOs to map API responses to known objects
* Add new Response to manage the Routes list
* Implement method to retrieve a list of Routes
* Add new Response to manage a single Route resource
* Implement method to retrieve a single Route
* Set ShowResponse as final
* Add new Response to manage the Create process
* Implement method to create a new Route
* Fix missing annotation
* Add new Response to manage the Delete Route process
* Implement method to delete a Route based on the ID
* Add new Response to manage the Update Route process
This response is based on Domain API docs due there are no examples on
Routes API docs. We may need to update the response.
* Implement method to update a Route based on the ID
* Require a $limit value greater than 0
* Require a $skip value greater than or equal to 0
* Set UpdateResponse as final
* Add new (empty) public methods to test the Routes API
* Provide method to get the Routes API from Mailgun Client
* Add missed annotation
* Update ShowResponse to return an instance of ApiResponse instead of the DTO
* Update annotation
* Fix annotation
* Update array $actions to provide an empty array by default
* Update parameters to make sure the last arg always is a DateTime (or null)
* Use empty()
* Remove DTO suffix
* Move DTOs to the parent folder/namespace
* Fix annotations
* Scrutinizer CI: Bug
The property deserializer does not seem to exist. Did you mean
serializer?
* Scrutinizer CI: Coding Style + Best Practice
It seems like you are loosely comparing two booleans. Considering using
the strict comparison === instead.
* Scrutinizer CI: Bug
It seems like $variables defined by parameter $variables on line ???
can also be of type null; however,
Mailgun\Messages\MessageBuilder::addRecipient() does only seem to
accept array, maybe add an additional type check?
* Scrutinizer CI: Documentation
The doc-type $class|SimpleResponse could not be parsed: Unknown type
name "$class" at position 0
Adding `object` instead of `SimpleResponse` due we have no files called
`SimpleResponse` in our project, and PhpDoc provides the `object`
keyword as a valid option.
https://www.phpdoc.org/docs/latest/guides/types.html#keywords
* Scrutinizer CI: Documentation
There is no parameter named $smtpPass. Did you maybe mean $smtpPassword?
* Scrutinizer CI: Unused Code
$resource is not used, you could remove the assignment.
* Scrutinizer CI: Unused Code
The parameter $message is not used and could be removed.
* Scrutinizer CI: Unused code
This method is not used, and could be removed. (x2)
* Scrutinizer CI: Patch
Doc Comments
* Scrutinizer CI: Patch
Doc comments
* Introduce a request builder.
We inject every dependency (eg RequestFactory and MultipartStreamBuilder) and we do not have to use postMultipart.
* code style
* Use uppercase on http verbs
* Added setters and use getters
* Added tests
* style
* Removed inheritence, Renamed classes, Mark all classes as final
* code style
* Updated docs
* Code style
* Be consistant with the naming of functions
* Fixed tests
* Added base for the new API
* code style
* Added response classes
* Added support for serializer
* The abstract API should not know of Mailgun
* Minor
* minor
* Using a client configrator
* code style
* Put HTTPClient in the configurator
* Do not use the api() function
* Use stable version of Assert
* style
* Fixed tests
* make the httpClient private
* Renamed ResponseSerializer to ResponseDeserializer
* Disabled tests that are testing error messages with Assert
* style fixes
* Refactoring fix
* Write test to verify expected behaviour when we provide a String
* Fix broken test checking if the variable is an Array
* PSR Standard using single quote instead of double quote
Adds the actual response message to the errors thrown on 400, 401 and
404 response codes. This provides a lot more useful info than the
current messages. The message doesn’t really give you much to go one. I
spent hours trying to find what I did wrong, double checking my API
keys and looking up the error on google.
* Changed constants to use new constant classes.
* Changed class docblock to include link to readme file
* Moved function descriptions of sendMessage() and verifyWebhookSignature() methods to function docblocks
