Improvements for Loopback Integrations

[zeerobug]

Hi Kimkha,

This is a fork for:

• Modified authClient to integrate AUTH_GET_PERMISSIONS case
• Added DELETE_MANY case for full compatibility with react-admin list view
• Added UPDATE_MANY (untested) There is a pending TODO to integrate Loopback side functions as hooks, which I can't do right now.

Thank you for your work !

[kimkha]

Thank you for your hard work and PR.

I have a question: why don't you just use default User model for authentication? Why must you create new model called Customer on your Loopback?

I think it's good for this plugin if we make this plugin simple: We have a default authentication which greatly integrate to default Loopback, and user can change to custom authentication url for themselves if they want.

[zeerobug]

Hi,

You are right, we have to keep it simple, I just copied and paste my function. I use customer because I needed to extend the default model. Please modify the README as you wish.

Cheers, Thomas

On 10 Jul 2018, at 17:23, Nguyễn Kim Kha [email protected] wrote:

Thank you for your hard work and PR.

I have a question: why don't you just use default User model for authentication? Why must you create new model called Customer on your Loopback?

I think it's good for this plugin if we make this plugin simple: We have a default authentication which greatly integrate to default Loopback, and user can change to custom authentication url for themselves if they want.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/kimkha/aor-loopback/pull/19#issuecomment-403862758, or mute the thread https://github.com/notifications/unsubscribe-auth/ASTa9fEMi5fvY1ksShWiL7MkO8wB9aNGks5uFMbjgaJpZM4VJC4r.