lorado

Results 18 comments of lorado

Hm, never noticed this behavior, but your proposal totally makes sense. @JasonTheAdams would you like to make a PR?

Ok, the change doesn't seem to be that hard. I'll try to find some time next week for a PR ;)

Just a question - do accessors (like `getPhoneNumberAttribute()`) count to "default resolver"? I hope the bug with double-calling will be fixed soon, anyway I don't want to lose this accessor-feature....

> `@lorado` Just a note about cleaning your schema, utilizing `@method` and `@rename` directive takes care of a lot of field resolvers. Using `@find`, `@paginate`, `@builder`, `@eq`, `@whereAuth` (recently added...

In my projects I handle such 'OR' logic directly in policies. For your example I would move logic of your abilities to the single ability "update". It works for me....

Could you try `@belongsToMany`? It works somehow, however I didn't look into the code, so don't know how it exactly works. You can currently fetch the data also without directive,...

> Does Eloquent already allow to do what we want through the relationship methods? I mean yes. We should probably do something like `$model->relation()->where('relation_primary_key', $args->id)->delete()`. Laravel adds the relation 'restriction'...

Sorry, I didn't notice, that markdown parsed \_\_ away, I edited the error, displayed by plugin. I guess you misunderstood me =) So I'll try to describe what is going...

@cmizzi Happy new year! Can you please merge this PR?

Oh well, it is kinda related to #12 If you integrate medialibrary, you could solve these two tickets