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.

Sign up for GitHub

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

Jump to bottom

Wrong route for Model.exists #260

Closed
2 of 6 tasks
viki53 opened this issue Dec 20, 2016 · 0 comments
Closed
2 of 6 tasks

Wrong route for Model.exists #260

viki53 opened this issue Dec 20, 2016 · 0 comments
Assignees
@viki53
Labels
2.1.0-rc.6 bug to be published
Milestone
2.1.0-rc.6

Comments

@viki53
Copy link
Contributor

viki53 commented Dec 20, 2016 •
edited
Loading

What type of issue are you creating?

  • Bug
  • Enhancement
  • Question

What version of this module are you using?

  • 2.0.10 (Stable)
  • 2.1.0-rc.5 (2.1 Release Candidate n)
  • Other

Please add a description for your issue:

When using the this.modelApi.exists() method, I got a 404 error, so I checked the actual URL called and the code and found out the BaseLoopBackApi had an error in the exists method:

In routeParams, exists should be :id/exists

I'll make a PR for this.
@viki53 viki53 mentioned this issue Dec 20, 2016
Merged
3 tasks
jonathan-casarrubias added a commit that referenced this issue Dec 20, 2016
@jonathan-casarrubias
Merge pull request #261 from viki53/patch-2
aa107d3 
Fixing issue #260
@jonathan-casarrubias jonathan-casarrubias added this to the 2.1.0-rc.6 milestone Dec 20, 2016
@viki53 viki53 mentioned this issue Dec 20, 2016
Merged
3 tasks
jonathan-casarrubias added a commit that referenced this issue Dec 20, 2016
@jonathan-casarrubias
Merge pull request #263 from mean-expert-official/master
da75e19 
Fixing issue #260 (dev)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@viki53 viki53

Labels
2.1.0-rc.6 bug to be published
Projects
None yet
Milestone
2.1.0-rc.6
Development

No branches or pull requests
2 participants
@jonathan-casarrubias @viki53