Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong ACL Name for deleting a record #252

Closed
jpbrown250 opened this issue Jan 17, 2017 · 3 comments
Closed

Wrong ACL Name for deleting a record #252

jpbrown250 opened this issue Jan 17, 2017 · 3 comments

Comments

@jpbrown250
Copy link

URL of the page

https://loopback.io/doc/en/lb2/Controlling-data-access.html

Nature of the issue

The ACL terms for how to define roles is inaccurate. The role for removing a record by ID is defined on the page as "destroyById" (which did not work). The one that does work for me is "deleteById".

Missing/inaccurate documentation content.

Suggested resolution

change the text of the documentation.

@crandmck
Copy link
Contributor

crandmck commented Jan 18, 2017

@jbrownssf Thanks for reporting the issue.

destroyById is an alias for deleteById - see https://github.com/strongloop/loopback/blob/master/lib/persisted-model.js#L359
They should both work, as should removeById.

There is currently a bug in our API generation code that prevents the API docs from showing the aliases.

If it's not working for you, could you please provide a test case?
@smartmouse Any thoughts on why destroyById might not work?

@crandmck
Copy link
Contributor

I'm going to move this issue to the loopback repo, because it's not a doc issue AFAICT.

@crandmck
Copy link
Contributor

Issue moved to strongloop/loopback #3133 via ZenHub

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants