restangular icon indicating copy to clipboard operation
restangular copied to clipboard

Support for Matrix URIs

Open HenriqueLimas opened this issue 11 years ago • 9 comments

It be cool if Restangular gives support for Matrix URIs. Maybe could be created a new Restangular methods "matrixParams" where we can define the matrix URIs.

Today:

Restangular.all('cambridge;scale=50000').post();

Purpose:

Restangular.all('cambridge').matrixParams({scale: 5000}).post()

Doc: http://www.w3.org/DesignIssues/MatrixURIs.html

HenriqueLimas avatar Dec 03 '14 18:12 HenriqueLimas

Hey! Great suggestion, to be supported in V2. If you feel powerful enough - please consider doing a PR for this particular feature :) Should not be that hard, it all boils down to exposing matrixParams() method that can be called any time on requestless Restangular object just to modify current route element.

grabbou avatar Feb 22 '15 13:02 grabbou

Yeah, I am going to make a PR. But, what is the branch I can do that? The "2.0-wip"?

HenriqueLimas avatar Feb 22 '15 21:02 HenriqueLimas

Thanks! Would love to see that feature :) please use wip/cleanup-improvements to fork and submit PR which is actually our development branch for now.

grabbou avatar Feb 22 '15 22:02 grabbou

Hi, I know it has been a long time, but trying to tidy a little bit, would still be interesting? Also, wouldn't it make more sense to use it as a configuration flag, so you define parameters same way and based on this parameter the final URL gets built as matrix URI or the existing one?

pedromarce avatar Feb 15 '16 20:02 pedromarce

@HenriqueLimas Is this still something you want to add?

daviesgeek avatar Jun 17 '16 23:06 daviesgeek

@daviesgeek, no nothing to add.

HenriqueLimas avatar Jun 18 '16 00:06 HenriqueLimas

@HenriqueLimas Sorry I should have been more clear. You have a PR up; is this still something that you would like to merge in? I'll work with you to merge it in if necessary.

daviesgeek avatar Jun 18 '16 00:06 daviesgeek

I think is something good to have, someone could want that. But I am not using it anymore..if you think is good to have that, we can merge or just close the pull request.

HenriqueLimas avatar Jun 18 '16 00:06 HenriqueLimas

Okay. This sounds like a good idea to add this, but I'm not familiar with it at all, so I won't merge right now, but will wait for other collaborators to check it out.

daviesgeek avatar Jun 18 '16 00:06 daviesgeek