-
Notifications
You must be signed in to change notification settings - Fork 5
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
Get method doesn't use existing resource #28
Comments
Hey @ryanmk54 sorry for the delay (I got sick over the weekend 🤒) But I guess it all started because you have a singular resource in I guess this is one more reason to figure out some way to work with singular resources 🤔 |
This was done to take advantage from the namespace features, that way if you do `api.namespace('users').get('profile')` it will work as expected. Fixes issue #28
Sorry about the delay, it should be fixed on #30 I will probably be merging it and releasing a new version next week 👍 |
I don't think it fixed it. I still get the same url as before. |
Weird 🤔 |
I changed package.json to
which resulted in the following yarn.lock entry after yarn install
Using |
So I investigated and my bad, I forgot to warn that just pointing to my PR would still load the old version because my PR didn't re-build the But I did the test with a personal project and seems to be working fine with the new code 😄 So I just published |
Awesome! Thanks. It worked. |
I have the route
current_sbm_employees_path | GET | /merchandising/sbm_employees/current(.:format)
I tried to access it with
api.resource('sbm_employees').get('current')
but that gives me
/merchandising/current
It works correctly if I use
api.resource('sbm_employees').list('current')
Based on the documentation it appears that accessing the url with the get method should work too.
The text was updated successfully, but these errors were encountered: