bavix / laravel-wallet

It's easy to work with a virtual wallet
https://bavix.github.io/laravel-wallet/
MIT License
1.12k stars 225 forks source link

conflict with laravel Cashier #87

Closed imhuso closed 5 years ago

imhuso commented 5 years ago

image image

rez1dent3 commented 5 years ago

@imhuso remove the CanPayFloat.

Upgrade:

$user->withdraw(...) to $user->wallet->withdraw(...)
$user->deposit(...) to $user->wallet->deposit(...)
$user->transfer(...) to $user->wallet->transfer(...)
$user->pay(...) to $user->wallet->pay(...)
$user->refund(...) to $user->wallet->refund(...)
...
imhuso commented 5 years ago

@rez1dent3 i can't understand.

rez1dent3 commented 5 years ago

@imhuso Do not use trait CanPayFloat. He creates this error. Where do you use shorthand syntax, use the full one.

Those if you use $user->pay(), then replace with $user->wallet->pay()

imhuso commented 5 years ago

@rez1dent3 Customer interface remove it together?

rez1dent3 commented 5 years ago

@imhuso Yes, it needs to be removed

imhuso commented 5 years ago

@rez1dent3 $user->wallet get null When I use the HasWalletFloat/ HasWallet trait then it work Let's do it first. thank :)

rez1dent3 commented 5 years ago

@imhuso Made changes, will be released soon.

The model will have two traits. https://github.com/bavix/laravel-wallet/commit/4bfedf8a16644a7f6a51c4b681456cf8da88b49a#diff-7801eb6c1e99b6063d7706e65ae2827e

Thanks for the information

rez1dent3 commented 5 years ago

@imhuso The release is ready, you can update the package. Tag: 3.1.6 Here are the changes: https://github.com/bavix/laravel-wallet/compare/3.1.5...3.1.6

imhuso commented 5 years ago

@rez1dent3 ok thanks

rez1dent3 commented 5 years ago

@imhuso There will be questions create a new issue