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

Rebased: Feature/promise based request interceptors #1433

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Rebased: Feature/promise based request interceptors #1433

wants to merge 2 commits into from

Conversation

lukeggchapman
Copy link

This PR is a rebase of #893 (from restangular 1.4.0 to ~1.5.2) and resolves issue #891

@szh
Copy link

szh commented Jan 2, 2017

This is already a few months old. Will it me merged?

@bostrom
Copy link
Collaborator

bostrom commented Jan 3, 2017

I'd have to have a proper look at the code, haven't had time yet though. If @lukeggchapman could do another rebase on top of master, then that would ease my burden.

@lukeggchapman
Copy link
Author

I started rebasing, but there has been a lot of whitespace changes in master which made the rebase more time consuming and I just don't have time to go through it at the moment.

@0xMatt
Copy link

0xMatt commented Apr 19, 2017

Considering the history of this PR it doesn't seem like we'll be getting it any time soon. Has anybody come up with alternative solutions for this problem?

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

Successfully merging this pull request may close these issues.

4 participants