stream-laravel icon indicating copy to clipboard operation
stream-laravel copied to clipboard

Is this package maintained anymore?

Open bryanbassett opened this issue 4 years ago • 6 comments

I am running the latest laravel and I even followed the demo projects examples for some tips. But when I now try to create a new item that belongsTo the user which hasMany of the item(s), I get this error:

BadMethodCallException with message 'Call to undefined method Illuminate\Database\Eloquent\Relations\BelongsTo::getKey()

If this package is no longer maintained I might just have to write the implementation myself instead of using the nice eloquent functionality it seems you all had worked hard to make.

On the other hand it could just be me, and that is also very likely. I have no idea why it won't work. I can create a new "item" if I dont use the ActivityTrait in the model.

bryanbassett avatar May 21 '21 14:05 bryanbassett

Yes, it's but this is possible since quite old at the moment. Luckily, revamp is coming soon. I will drop an update here by Monday.

ferhatelmas avatar May 21 '21 15:05 ferhatelmas

image

I was able to solve my issue by using these overrides. Overflows? Idk what they are technically called. But yeah. Solved my issue at least. I still think this should not be necessary however. I think the problem was that get_class($instance) was returning BelongsTo:: instead of the actual class name of the model.

bryanbassett avatar May 21 '21 16:05 bryanbassett

@ferhatelmas Hello any news about a package update (was supposed to be posted in May) ? Thanks, Denis

dgillier avatar Jul 29 '21 07:07 dgillier

Yes, it's but this is possible since quite old at the moment. Luckily, revamp is coming soon. I will drop an update here by Monday.

That Monday passed a 5 months ago...

nezaboravi avatar Oct 12 '21 12:10 nezaboravi

Any update ?

dgillier avatar Nov 23 '21 08:11 dgillier

FYI, there is a new release https://github.com/GetStream/stream-laravel/releases/tag/2.4.0

ferhatelmas avatar Dec 21 '21 11:12 ferhatelmas