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

Brainstem should support options to refresh/clear request parameters #77

Open
naganowl opened this issue Apr 28, 2016 · 2 comments
Open
Milestone

Comments

@naganowl
Copy link
Contributor

There are use cases where we are making requests (usually filters) where the parameters are additive and it be nice to have Brainstem add the params to the last fetch options automatically (via a deep merge). We might be able to leverage the existing refresh call and/or make that an option.

On the flip side, we also want the ability to blow away all the parameters (except the ones specified) in case we are performing a reset/clear of the request.

For backwards compatibility, this could just be a refresh to trigger between making incoming options additive vs. replacement.

@naganowl naganowl added this to the 0.5 milestone Apr 28, 2016
@juanca
Copy link
Contributor

juanca commented Apr 28, 2016

What about options similarly named to backbone options, e.g. add, remove, reset type of deal?

@naganowl
Copy link
Contributor Author

Probably could use reset: true for flushing the options and reset: false for adding them on.

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

No branches or pull requests

2 participants