Need accept null in return types

[sahartak]

return types generating error for null values, for example in class Slince\Shopify\Model\Orders\Order public function getFulfillmentStatus(): string { return $this->fulfillmentStatus; }

need change return types to public function getFulfillmentStatus(): ?string {}

[slince]

hello, can you submit a pr for this?

[dashko]

This is an issue for us as well.

[happyclicker]

This. needs to be done pretty much everywhere - Shopify returns a null when it feels like it instead of a "". Address, Order, Customer, everything.

[slince]

fixed in 3.0.2

[dashko]

This was not fixed in 3.0.2 @slince :

Return value of Slince\Shopify\Model\Orders\Order::getShippingAddress() must be an instance of Slince\Shopify\Model\Orders\Common\OrderAddress, null returned

[slince]

@dashko fixed